groombook/app
Archived
13
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat(GRO-106): portal Communication tab — real backend #412

Closed
lint-roller-qa[bot] wants to merge 4 commits from feat/GRO-106-portal-communication-real into dev
pull from: feat/GRO-106-portal-communication-real
merge into: groombook:dev
Conversation 1 Commits 4 Files Changed 26
+4045 -6780

4 Commits

Author SHA1 Message Date
Chris Farhood 389c10fe92 fix(GRO-986): add businessId scoping to portal conversation messages query 
The GET /portal/conversation/messages endpoint was missing businessId
scoping, allowing cross-tenant data access. This adds businessId from
businessSettings to the conversation lookup in the messages endpoint,
matching the existing GET /portal/conversation protection.

Also adds missing lt mock to portal test suite.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
 2026-05-14 18:22:06 +00:00
Chris Farhood d2291e3a4a feat(GRO-106): staff messages page 
- Adds staff conversations API (GET /api/conversations, GET /api/conversations/:id/messages, POST /api/conversations/:id/messages) with auth scoping and cross-tenant protection
- Adds staffReadAt column to conversations table for unread tracking
- Adds staff Messages page with two-column inbox layout (thread list + conversation view + composer)
- Adds Messages entry to staff sidebar navigation
- Includes tests for the MessagesPage component

Part of GRO-106 (SMS/MMS integration) Phase 1.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-14 10:27:06 +00:00
Chris Farhood fbf3527085 fix(GRO-1215): resolve ESLint error, cursor pagination, and UAT playbook gaps 
- Add and() + lt() imports from @groombook/db
- Apply businessId to conversation WHERE clause for cross-tenant isolation
  (GET /portal/conversation: clientId AND businessId both scoped)
- Fix cursor pagination: apply lt(messages.createdAt, cursorMsg.createdAt)
  to the cursor WHERE clause so pages actually paginate
- Add UAT_PLAYBOOK.md §4.9.1 Communication tab test cases:
  TC-APP-4.9.6 message history with conversation
  TC-APP-4.9.7 empty state (no conversation yet)
  TC-APP-4.9.8 composer disabled with tooltip
  TC-APP-4.9.9 cross-tenant isolation

Co-Authored-By: Paperclip <noreply@paperclip.ing>
 2026-05-14 09:53:42 +00:00
Chris Farhood 5f717c4467 feat(GRO-106): portal Communication tab — real backend 
- Added GET /portal/conversation and GET /portal/conversation/messages endpoints
- Created Communication.api.ts with typed fetchers and React hooks
- Rewired Communication.tsx to use real API, removed mock data
- Added composer-disabled bar with "Reply from your phone" tooltip
- Added conversation route tests to portal.test.ts

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
 2026-05-14 06:07:01 +00:00