-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Gary add materials page #577
Conversation
This pr works as expected check #1427 for testing details |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as described, left my review in the front end PR. Great job!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @GaryB93
I left the review for BE in the FE part, these features works well. Thank you for the nice work!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code looks good to me! Great job @tdkent! Also tested FE part and it works as expected!
Description
This pull request adds the backend logic for the Purchase Request: Material form in the BM Dashboard (Phase 2).
Related PRS (if any):
This PR must be checked out to test #1427 frontend PR
Main changes explained:
How to test:
git checkout gary_add_materials_page
npm install
npm run dev
Notes