Skip to content

247 - Missing endpoints in donors#286

Open
mehanana wants to merge 2 commits into
mainfrom
247-donors-missing-crud-endpoints
Open

247 - Missing endpoints in donors#286
mehanana wants to merge 2 commits into
mainfrom
247-donors-missing-crud-endpoints

Conversation

@mehanana

Copy link
Copy Markdown
Contributor

ℹ️ Issue

Closes #247

📝 Description

Added missing endpoints to donors lambda

  • POST /donors
  • DELETE /donors/{id}
  • DELETE /donations/{id}

Briefly list the changes made to the code:

  1. Added the new endpoints to the donors handler
  2. Wrote tests for the new endpoints & tested on swagger UI

✔️ Verification

Ran jest test suite & tested the endpoints on swagger ui

Jest tests:
Screenshot 2026-07-18 234305

Unauthorized:
Screenshot 2026-07-18 235105
Screenshot 2026-07-18 235118

200 success:
Screenshot 2026-07-18 235733
Screenshot 2026-07-18 235800
Screenshot 2026-07-19 000342

400 errors:
Screenshot 2026-07-18 235815
Screenshot 2026-07-18 235827
Screenshot 2026-07-18 235931

🏕️ (Optional) Future Work / Notes

Did you notice anything ugly during the course of this ticket? Any bugs, design challenges, or unexpected behavior? Write it down so we can clean it up in a future ticket!

@mehanana
mehanana requested a review from nourshoreibah as a code owner July 19, 2026 04:18
@github-actions
github-actions Bot requested a review from tsudhakar87 July 19, 2026 04:18
github-actions Bot added a commit that referenced this pull request Jul 19, 2026
@nourshoreibah nourshoreibah added the test-environment Creates a temporary (nearly free) test environment. Uses prod DB and cognito label Jul 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🌿 ⏳ Creating preview environment… (logs)

@github-actions

Copy link
Copy Markdown
Contributor

🌿 Preview environment — ready ✅

Open: https://d3nmtjoh6ir9ym.cloudfront.net/pr-286/
API: https://wnse841414.execute-api.us-east-2.amazonaws.com/prod

Shared RDS + Cognito (prod data); DB migrations are not applied here. New commits update this environment in place — a note is posted here on each update. Remove the test-environment label or close the PR to tear it down.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test-environment Creates a temporary (nearly free) test environment. Uses prod DB and cognito

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Backend] Missing CRUD endpoints across lambdas

2 participants