Skip to content

feat: GET invoices/[id]/messages (Issue #375)#394

Open
HuiNeng6 wants to merge 1 commit intodavedumto:mainfrom
HuiNeng6:feat/routes-b-invoice-messages-get
Open

feat: GET invoices/[id]/messages (Issue #375)#394
HuiNeng6 wants to merge 1 commit intodavedumto:mainfrom
HuiNeng6:feat/routes-b-invoice-messages-get

Conversation

@HuiNeng6
Copy link
Copy Markdown

Closes #375

@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 HuiNeng6 force-pushed the feat/routes-b-invoice-messages-get branch from c8def4d to 5e95fe5 Compare March 27, 2026 21:13
- Returns all messages on an invoice's thread
- Ordered chronologically (createdAt asc)
- Verifies invoice ownership before returning messages
- Returns empty array when no messages exist (not 404)
- Returns 403 if invoice belongs to different user
- Returns 404 if invoice does not exist
- Returns 401 for unauthenticated requests

Addresses Issue davedumto#422
@HuiNeng6 HuiNeng6 force-pushed the feat/routes-b-invoice-messages-get branch from 5e95fe5 to a3c23db Compare March 27, 2026 21:16
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/invoices/[id]/messages — list messages on an invoice

1 participant