-
Notifications
You must be signed in to change notification settings - Fork 0
[SSF-134]: Update FM Management Frontend #104
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
base: main
Are you sure you want to change the base?
Conversation
| @@ -42,46 +49,30 @@ export class DonationsController { | |||
| type: 'object', | |||
| properties: { | |||
| foodManufacturerId: { type: 'integer', example: 1 }, | |||
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.
NOTE FOR PR Reviewer: We deleted this since it does not make sense to have either of these fields here in creating a donation. The status will always be the same, as will the date donated.
dburkhart07
left a comment
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.
Leaving comment for PR reviewer
ℹ️ Issue
Closes #134
📝 Description
✔️ Verification
Took screenshots of the pages:



Verified each updated endpoint worked on Postman
🏕️ (Optional) Future Work / Notes
We will need to update this later on when we go to finish up the Action Required tab.