SSF-219 New Donation Form Required Fields Backend Updates#190
Conversation
dburkhart07
left a comment
dburkhart07
left a comment
There was a problem hiding this comment.
Looks good, just missed a few spots.
Here are a few others that I think we can make sure we update the fact that these are now required in a few other spots in the backend:
- donationItems entity
- manufacturer service getFmDonations function
- donation-details-dto
- We also need to write a migration to reflect these changes in the DB
Yurika-Kan
left a comment
Yurika-Kan
left a comment
There was a problem hiding this comment.
LGTM. This is awesome. Thanks Justin!
Yurika-Kan
left a comment
Yurika-Kan
left a comment
There was a problem hiding this comment.
actually, could you update the confirm item details page since it is a similar ui?
can we match the required fields logic by having asterisks for the fields & grayed out submit button functionality when some fields are empty. let's ensure there are no workaround ways to submit "partial" donation items with empty required fields!
reword:
Please fill out the missing fields information to record donation details.
to:
Please confirm the following information to record donation details.
Please do not include shipping/delivery costs in Food Donation Value.
add (Fair Market Value of Food Only) to Donation Value
dburkhart07
left a comment
dburkhart07
left a comment
There was a problem hiding this comment.
one small thing!! tysm justin <3333
Yurika-Kan
left a comment
Yurika-Kan
left a comment
There was a problem hiding this comment.
approving - one small thing but LGTMMMM
3 participants
ℹ️ Issue
Closes https://vidushimisra.atlassian.net/browse/SSF-219
📝 Description
Made it so oz. per item, donation value, and food rescue are required not nullable fields in backend dtos and updated tests.
Frontend: Minor visual changes to reflect these required fields (details in ticket)
✔️ Verification
Verified frontend design and tests passed in backend.
🏕️ (Optional) Future Work / Notes
N/A