Skip to content

feat(frontend): harden banks API and instrument status telemetry - #1288

Merged
github-actions[bot] merged 5 commits into
leojay-net:mainfrom
Anthony-19:feat/add_request_validation
Jul 26, 2026
Merged

feat(frontend): harden banks API and instrument status telemetry#1288
github-actions[bot] merged 5 commits into
leojay-net:mainfrom
Anthony-19:feat/add_request_validation

Conversation

@Anthony-19

Copy link
Copy Markdown
Contributor

Summary

  • validate /api/banks query parameters with Zod and return a machine-readable INVALID_REQUEST error on invalid input
  • apply the shared rate limiter to /api/banks, with unit coverage for quota rejection
  • emit consent-aware structured telemetry for Paystack payment status and browser connectivity transitions
  • add make ci to run the frontend and contract CI matrix locally

Rationale

Contributors previously had to reconstruct checks from several workflow files. make ci provides one local entry point covering the same validation categories: frontend typecheck, lint, build, coverage, E2E, and contract test/build/clippy.

Testing

  • git diff --check
  • Added banks API and status telemetry unit tests
  • pnpm typecheck, pnpm lint, and pnpm test:unit could not be executed locally because dependency installation exceeded the environment time limit before completion.

Closes #1276
Closes #1265
Closes #1206
Closes #1205

@drips-wave

drips-wave Bot commented Jul 26, 2026

Copy link
Copy Markdown

@Anthony-19 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@github-actions
github-actions Bot merged commit 0d6baf3 into leojay-net:main Jul 26, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant