Skip to content

Sk/ssf 155 close food requests endpoint#145

Open
swarkewalia wants to merge 12 commits intomainfrom
sk/SSF-155-close-food-requests-endpoints
Open

Sk/ssf 155 close food requests endpoint#145
swarkewalia wants to merge 12 commits intomainfrom
sk/SSF-155-close-food-requests-endpoints

Conversation

@swarkewalia
Copy link
Copy Markdown

@swarkewalia swarkewalia commented Apr 2, 2026

ℹ️ Issue

Closes 155

📝 Description

  • Added PATCH /requests/:requestId/close endpoint for volunteers to close active food requests
  • Closing a request only updates request status and does not affect associated order statuses

✔️ Verification

tested endpoint using swagger + service tests

@swarkewalia swarkewalia changed the title Sk/ssf 155 close food requests endpoints Sk/ssf 155 close food requests endpoint Apr 2, 2026
@Yurika-Kan Yurika-Kan self-requested a review April 2, 2026 06:07
Copy link
Copy Markdown

@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.

Can we just add a short controller test in .spec file but besides that 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