Skip to content

[SSF 215] - Audit unused endpoints#177

Merged
dburkhart07 merged 5 commits into
mainfrom
ddb/SSF-215-audit-unused-endpoints
May 28, 2026
Merged

[SSF 215] - Audit unused endpoints#177
dburkhart07 merged 5 commits into
mainfrom
ddb/SSF-215-audit-unused-endpoints

Conversation

@dburkhart07
Copy link
Copy Markdown

ℹ️ Issue

Closes #215

📝 Description

Removed all unused endpoints and service functions

✔️ Verification

  • Ensured all tests passed
  • Smoke tested pages

🏕️ (Optional) Future Work / Notes

N/A

Comment thread apps/backend/src/donations/donations.controller.ts
Copy link
Copy Markdown
Collaborator

@Yurika-Kan Yurika-Kan left a comment

Choose a reason for hiding this comment

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

wowowow we are so cleaning!

can we also delete app.controller.ts, app.controller.spec.ts, app.service.ts and this endpoint /users/:id/stats (frontend renders based on user)? + check comment on a endpoint that is actually used

@dburkhart07 dburkhart07 requested a review from Yurika-Kan May 24, 2026 19:22
@Yurika-Kan Yurika-Kan self-assigned this May 25, 2026
Copy link
Copy Markdown
Collaborator

@Yurika-Kan Yurika-Kan left a comment

Choose a reason for hiding this comment

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

keep request, order, and donation patch & deletes to support our in progress object lifecycle functionalities!

otherwise 🧽 🫧 cleaned

Comment thread apps/backend/src/donations/donations.controller.ts
Comment thread apps/backend/src/foodRequests/request.controller.ts
Comment thread apps/backend/src/orders/order.controller.ts
Copy link
Copy Markdown
Collaborator

@Yurika-Kan Yurika-Kan left a comment

Choose a reason for hiding this comment

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

LGTM thanks!!! 🔥 🗑️

Copy link
Copy Markdown

@jxuistrying jxuistrying left a comment

Choose a reason for hiding this comment

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

lgtm!

@dburkhart07 dburkhart07 merged commit 38ccec0 into main May 28, 2026
4 checks passed
@dburkhart07 dburkhart07 deleted the ddb/SSF-215-audit-unused-endpoints branch May 28, 2026 04:25
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