Skip to content

Pantry Order Management Page and Confirm Delivery Modal#116

Open
Juwang110 wants to merge 22 commits intomainfrom
jw/order-management-homepage
Open

Pantry Order Management Page and Confirm Delivery Modal#116
Juwang110 wants to merge 22 commits intomainfrom
jw/order-management-homepage

Conversation

@Juwang110
Copy link

@Juwang110 Juwang110 commented Feb 25, 2026

ℹ️ Issue

Closes https://vidushimisra.atlassian.net/jira/software/projects/SSF/boards/1/backlog?jql=assignee%20%3D%20712020%3A10ef9ad9-e290-4bbd-8c4b-cb215e8d449a&selectedIssue=SSF-142

📝 Description

I refactored the frontend for the pantry order management page to match the Figma design. I also implemented the confirm order delivery modal, which features calling the confirmDelivery route. This modal is a form that involves uploading pictures to send.

Notable frontend route: /pantry-order-management

✔️ Verification

I verified the frontend matched the figma design. I also verified confirm delivery works as expected with making sure the date, feedback and pictures show up in the database.

Screenshot 2026-02-25 152926 image

🏕️ (Optional) Future Work / Notes

Notes: The assignee functionality is purposefully not present.

@sam-schu sam-schu self-requested a review February 27, 2026 15:05
Copy link
Collaborator

@sam-schu sam-schu left a comment

Choose a reason for hiding this comment

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

  1. It looks like we're missing Filter, unless Priya said we don't need that?
  2. For the modals, can we create the component once and just vary the data passed in/displayed depending on which order is selected, rather than creating one modal instance per order?

@Juwang110
Copy link
Author

Juwang110 commented Mar 9, 2026

  1. It looks like we're missing Filter, unless Priya said we don't need that?
  2. For the modals, can we create the component once and just vary the data passed in/displayed depending on which order is selected, rather than creating one modal instance per order?
  1. Priya said we don't need filter here
  2. Done!

@Juwang110 Juwang110 requested a review from sam-schu March 9, 2026 17:43
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.

3 participants