Skip to content

[SSF-154] Endpoint for Volunteers to Retrieve All Requests for Assigned Pantries#124

Open
amywng wants to merge 4 commits intomainfrom
acw/SSF-154-volunteer-requests
Open

[SSF-154] Endpoint for Volunteers to Retrieve All Requests for Assigned Pantries#124
amywng wants to merge 4 commits intomainfrom
acw/SSF-154-volunteer-requests

Conversation

@amywng
Copy link
Member

@amywng amywng commented Mar 9, 2026

ℹ️ Issue

Closes SSF-154

📝 Description

  • added endpoint for volunteers to get requests associated with assigned pantries
  • wrote tests for endpoint
  • made frontend page to test endpoint (my computer was acting up so... yeah. it's p much just adminDonation page copied w/ fixed variables tho) http://localhost:4200/volunteer-request-management

✔️ Verification

  • verified tests pass
  • tested on postman
  • verified frontend page
Screenshot 2026-03-09 at 12 19 20 AM

🏕️ (Optional) Future Work / Notes

reviewer: might be easier to wait for these tickets to be merged (since the request tests are from this pr and the homepage section is from this pr
also the tests aren't going to pass atm bc of the above prs ^

@dburkhart07 dburkhart07 self-requested a review March 9, 2026 22:35
Copy link

@dburkhart07 dburkhart07 left a comment

Choose a reason for hiding this comment

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

just a few nits!

Copy link

@dburkhart07 dburkhart07 left a comment

Choose a reason for hiding this comment

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

few more small things on the frontend side.

@amywng amywng requested a review from dburkhart07 March 10, 2026 03:38
Copy link

@dburkhart07 dburkhart07 left a comment

Choose a reason for hiding this comment

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

lgtm!! 🙏 🚶‍♀️ 🎥 🌃

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.

2 participants