Backend Integration PR#2
Open
AaronCrvl wants to merge 14 commits into
Open
Conversation
…ervice patterns and routes - Implemented stock selection controls and integration - Added backend API calls for stock transactions - Added backend API calls for stock products - Updated services patterns and routes
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds stock selection integration and backend API calls for stock-related operations. The changes focus on standardizing service patterns and implementing proper backend integration for stock transactions and products.
- Refactored service function names to follow PascalCase naming convention
- Standardized error handling by using
toast.error()instead oftoast() - Integrated backend API calls for stock transactions and product operations
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| src/services/api/user.services.js | Added route constant for standardized URL construction |
| src/services/api/stockTransaction.services.js | Renamed functions to PascalCase, updated API endpoints, and improved error handling |
| src/services/api/stockProduct.services.js | Renamed functions to PascalCase and standardized error handling |
| src/services/api/stockClosure.services.js | Updated error handling to use toast.error() |
| src/services/api/stock.service.js | Added new SetSessionStock function and renamed existing function |
| src/services/api/auth.services.js | Added route constant for URL construction |
| src/pages/StockTransaction.jsx | Integrated backend API calls with proper error handling |
| src/pages/Home.jsx | Fixed map function parameter order |
| src/pages/Dashboard.jsx | Integrated backend API calls for product operations |
| src/components/StockControl.jsx | Added session stock management and improved stock selection |
- fix on Home.jsx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- fix in StockControl Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- fix in StockControl.jsx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- Fix in stockTransaction.services.js Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…d user management - Integrated stock closure functionality with backend system. - Applied standardized code comment patterns for improved clarity. - Updated service route definitions for better modularity and routing control. - Developed user account management features, including create, update, and deactivate functionality
Owner
Author
|
AaronCrvl
changed the title
- Refactored the initialization of itensGridView to use availableItensList when available. - Updated item addition and search logic to fallback to availableItensList if availableProducts is empty, improving robustness and preventing errors when product sources are missing.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…ervice patterns and routes