Skip to content

feat: add recall list endpoints for compliance dashboard#3610

Merged
TaprootFreak merged 2 commits intodevelopfrom
feat/recall-list
Apr 23, 2026
Merged

feat: add recall list endpoints for compliance dashboard#3610
TaprootFreak merged 2 commits intodevelopfrom
feat/recall-list

Conversation

@TaprootFreak
Copy link
Copy Markdown
Collaborator

Summary

  • Adds GET /recall (list all) and GET /recall/:id (single) on RecallController
  • Eager-loads bankTx, checkoutTx and user relations so the compliance dashboard receives full context without extra calls
  • Admin-only guard stack matches existing POST/PUT routes

Test plan

  • GET /recall as ADMIN → 200, array with all recalls incl. relations
  • GET /recall/:id valid id → 200, single recall with relations
  • GET /recall/:id invalid id → 404
  • GET endpoints reject non-ADMIN users

Adds GET /recall and GET /recall/:id returning recalls with eager-loaded
bankTx, checkoutTx and user relations. Both routes are admin-only,
consistent with existing POST/PUT.
Switches RecallController guards from ADMIN to COMPLIANCE so compliance
operators can use the dashboard end-to-end (list, create, update).
ADMIN and SUPER_ADMIN still have access via additional roles.
@TaprootFreak TaprootFreak merged commit 0adf91f into develop Apr 23, 2026
7 checks passed
@TaprootFreak TaprootFreak deleted the feat/recall-list branch April 23, 2026 09:24
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.

1 participant