Skip to content

[SSF 229] - Update Pantry Food Request Form#197

Merged
dburkhart07 merged 3 commits into
mainfrom
ddb/SSF-229-update-pantry-food-request-form
Jun 28, 2026
Merged

[SSF 229] - Update Pantry Food Request Form#197
dburkhart07 merged 3 commits into
mainfrom
ddb/SSF-229-update-pantry-food-request-form

Conversation

@dburkhart07

Copy link
Copy Markdown

ℹ️ Issue

Closes #229

📝 Description

  • Made migration to support all backend updates (fixed enum, 2 new fields for food request)
  • Updated frontend page to reflect these in both the new food request form, and food request details modal

✔️ Verification

  • Verified all tests passed
  • Smoke tested new page
image

🏕️ (Optional) Future Work / Notes

Will need to verify that new fields are editable pending #186 getting merged

@Yurika-Kan Yurika-Kan self-assigned this Jun 20, 2026

@Juwang110 Juwang110 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Two minor things

Comment thread apps/frontend/src/components/forms/requestFormModal.tsx Outdated
Comment thread apps/frontend/src/components/forms/requestDetailsModal.tsx Outdated

@jiang-h-y jiang-h-y left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! checked new enums, tests/dummy data, and migration

@Yurika-Kan Yurika-Kan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMMM! tested & looks awesomely updated

@dburkhart07 dburkhart07 merged commit aca9282 into main Jun 28, 2026
4 checks passed
@dburkhart07 dburkhart07 deleted the ddb/SSF-229-update-pantry-food-request-form branch June 28, 2026 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants