Skip to content

feat: implement GET /api/routes-b/search endpoint (Issue #374)#386

Open
HuiNeng6 wants to merge 2 commits intodavedumto:mainfrom
HuiNeng6:feat/routes-b-search
Open

feat: implement GET /api/routes-b/search endpoint (Issue #374)#386
HuiNeng6 wants to merge 2 commits intodavedumto:mainfrom
HuiNeng6:feat/routes-b-search

Conversation

@HuiNeng6
Copy link
Copy Markdown

Summary

Implements Issue #374 - GET /api/routes-b/search

Features

  • Search invoices and bank accounts by query string
  • Query param q required, minimum 2 characters
  • Type param filters to invoices or bank-accounts
  • Both queries run in parallel

Acceptance Criteria

  • Returns 200 with both result arrays
  • Returns 400 if q is missing or under 2 chars
  • ?type=invoices skips bank account query
  • ?type=bank-accounts skips invoices query
  • Invalid type value returns 400
  • Both queries run in parallel
  • Returns 401 for unauthenticated requests

Closes #374

- Search invoices and bank accounts by query string
- Query param q required, minimum 2 characters
- Type param filters to invoices or bank-accounts
- Both queries run in parallel

Closes davedumto#374
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 27, 2026

@HuiNeng6 is attempting to deploy a commit to the david's projects Team on Vercel.

A member of the Team first needs to authorize it.

@HuiNeng6
Copy link
Copy Markdown
Author

@davedumto Ready for review and merge! 🙏

@davedumto
Copy link
Copy Markdown
Owner

hi fix conflicts please

@HuiNeng6
Copy link
Copy Markdown
Author

@davedumto Conflicts have been resolved. Ready for merge! 🙏

@HuiNeng6
Copy link
Copy Markdown
Author

@davedumto Following up — PR is mergeable and ready. Any update on when this can be merged? 🙏

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.

[routes-b] GET /api/routes-b/search — search across invoices and bank accounts

2 participants