Skip to content

[SSF 195] - FM Required Actions Frontend#166

Open
dburkhart07 wants to merge 4 commits intomainfrom
ddb/SSF-195-fm-required-actions-frontend
Open

[SSF 195] - FM Required Actions Frontend#166
dburkhart07 wants to merge 4 commits intomainfrom
ddb/SSF-195-fm-required-actions-frontend

Conversation

@dburkhart07
Copy link
Copy Markdown

@dburkhart07 dburkhart07 commented Apr 26, 2026

ℹ️ Issue

Closes #195

📝 Description

Backend changes:
I realized during development that the single order updating was deprecated, as we needed a way to update the tracking link and shipping cost for all of the orders at once. Therefore, I added a new endpoint and service function (as well as tests) for this, and got rid of the old one. The backend now takes in ether the shipping cost or tracking link for each order from the donation, and updates the order to shipped when both are provided eventually. Additionally, I also realized now that we would need to make it so that we can edit fields for confirming a donation item, even after it gets confirmed. I modularized the old confirm donation item details endpoint to accept all fields as optional, and rather just serve as an update function that, in the even where all fields got values, we would make it so the details became confimred.

Frontend changes:
I made the frontend page, with a few minor adjustments agreed upon with design:

  1. We get all pending orders at the start. A FM can update the shipping cost and tracking link for these however often they want, but only when all pending orders have been marked shipped will this part of the required actions be considered done. This is accomplished by adding shipping cost and tracking link for all orders in the end, until then, the message remains.
  2. For donation items, we now show every item for the donation every single time. The user can then update existing donation items' details, or add to their own. In the end, after all the donation item have a field, all our items are considered confirmed, which is also part of the completed actions process.

✔️ Verification

  • Tested endpoints in Postman

Screenshots of pages:
image
image
image

🏕️ (Optional) Future Work / Notes

N/A

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.

1 participant