Skip to content

feat: implement POST /api/routes-b/invoices/[id]/messages endpoint (Issue #378)#384

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

feat: implement POST /api/routes-b/invoices/[id]/messages endpoint (Issue #378)#384
HuiNeng6 wants to merge 1 commit intodavedumto:mainfrom
HuiNeng6:feat/routes-b-invoice-messages-post

Conversation

@HuiNeng6
Copy link
Copy Markdown

Summary

Implements Issue #378 - POST /api/routes-b/invoices/[id]/messages

Features

  • Add message to invoice thread
  • Content validation: required, non-empty, max 1000 chars
  • Verify invoice ownership
  • Returns 201 with created message

Acceptance Criteria

  • Returns 201 with created message
  • Content over 1000 chars returns 400
  • Missing or empty content returns 400
  • Returns 403 if invoice belongs to a different user
  • Returns 404 if invoice does not exist
  • Returns 401 for unauthenticated requests

Closes #378

…ssue davedumto#378)

- Add message to invoice thread
- Content validation: required, non-empty, max 1000 chars
- Verify invoice ownership
- Return 201 with created message

Closes davedumto#378
@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

Hi @davedumto! 👋 Following up on this PR. Ready for review. Thank you!

@HuiNeng6
Copy link
Copy Markdown
Author

@davedumto This PR is ready for review. All endpoints implemented as specified. Ready to merge! 🙏

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] POST /api/routes-b/invoices/[id]/messages — add a message to an invoice

1 participant