Skip to content

[SSF-164] all requests endpoint and add donation tests#129

Merged
amywng merged 6 commits intomainfrom
acw/SSF-164-food-request-endpoint
Mar 19, 2026
Merged

[SSF-164] all requests endpoint and add donation tests#129
amywng merged 6 commits intomainfrom
acw/SSF-164-food-request-endpoint

Conversation

@amywng
Copy link
Member

@amywng amywng commented Mar 16, 2026

ℹ️ Issue

Closes SSF-164

📝 Description

  • added getAllFoodRequests to controller
  • added getAll to service
  • added controller/service tests
  • added missing donation tests and fixed some things in donation files

✔️ Verification

  • tested on Postman and verified written tests passed

@amywng amywng changed the title all requests endpoint and add donation tests [SSF-164] all requests endpoint and add donation tests Mar 16, 2026
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.

thanks for the donation tests lol LGTM after this!

@amywng amywng requested review from Juwang110 and sam-schu March 19, 2026 03:06
Copy link

@Juwang110 Juwang110 left a comment

Choose a reason for hiding this comment

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

Looks great!!

@amywng amywng merged commit de32fbf into main Mar 19, 2026
4 checks passed
@amywng amywng deleted the acw/SSF-164-food-request-endpoint branch March 19, 2026 14:09
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