chore: promote dev → uat (GRO-1794 booking analytics) #27

Merged

The Dogfather merged 6 commits from dev into uat 2026-05-26 13:16:40 +00:00

Conversation 0 Commits 6 Files Changed 12

+364 -46

The Dogfather commented 2026-05-26 13:16:24 +00:00

Member

Copy Link

Copy Source

Summary

Promotes dev to uat after merging PR #24 (booking funnel analytics events).

• 6 analytics events wired in public + portal booking flows
• Thin window.dispatchEvent wrapper, no SDK dependency
• No PII in payloads
• Unit tests + UAT playbook included

cc @cpfarhood

## Summary Promotes dev to uat after merging PR #24 (booking funnel analytics events). - 6 analytics events wired in public + portal booking flows - Thin window.dispatchEvent wrapper, no SDK dependency - No PII in payloads - Unit tests + UAT playbook included cc @cpfarhood

The Dogfather added 6 commits 2026-05-26 13:16:24 +00:00

feat(GRO-1792): add recovery paths to booking error and cancellation pages ... 344a32e3e4

- Add "Start a new booking" button to BookingError linking to /admin/book
- Add "Book again" button to BookingCancelled linking to /admin/book
- Add business contact info section to BookingError (from BUSINESS_CONTACT_INFO constant)
- Replace hardcoded colors with CSS variables (--color-error, --color-cancelled, etc.)
- Add page-level string constants to eliminate hardcoded strings
- Add unit tests for both pages (9 tests passing)

Co-Authored-By: Paperclip <noreply@paperclip.ing>

feat(GRO-1794): add booking funnel analytics events ... 2e99ed520f

- New analytics utility (src/lib/analytics.ts) with ANALYTICS_EVENTS constants
  and fireAnalyticsEvent() – thin wrapper over window.dispatchEvent, no-op safe
  Built for Plausible/GTM integration later.

- Public booking wizard (Book.tsx): fires step-transition events at each step
  (service → time → contact → submit) plus booking_confirmed on the dedicated
  confirmation page.

- Portal BookingFlow (Appointments.tsx): fires equivalent events for the
  portal booking flow. booking_confirmed fires via useEffect when the inline
  success state is shown.

- BookingErrorPage: fires booking_error on mount (no PII in payload).

Events include step name and flow type (public/portal) but contain no PII:
no names, emails, phone numbers, or pet names in any payload.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

docs(UAT_PLAYBOOK.md): add §5.24 booking funnel analytics test cases ... 3bccb1ac01

Gro-1794 required UAT test cases for the booking funnel analytics events.
Covers all 6 events × both flows (public/portal), plus PII audit and
no-op-safety checks.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

fix: add missing vi import and fix getByText exact match assertions ... 7e5a851d9c

- analytics.test.ts: add vi to vitest import (was used at lines 24, 37, 66)
- BookingError.test.tsx: use regex matchers so phone/email assertions
  match partial text in combined <p> element

Co-Authored-By: Paperclip <noreply@paperclip.ing>

fix: add non-null assertion on listener.mock.calls[0] (TS strict mode) ... 112c61ab1c

Lines 28 and 40 access mock.calls[0] which is possibly undefined under
strict TypeScript. Adding ! to satisfy TS2532.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Merge pull request 'feat(GRO-1794): booking funnel analytics events' (#24) from feature/gro-1165d-booking-analytics into dev ... 33a1b3ed7a

Merge PR #24: feat(GRO-1794): booking funnel analytics events

CTO-approved. QA passed. All CI green.

The Dogfather merged commit 87939e5413 into uat 2026-05-26 13:16:40 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something isn't working

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

good first issue

Good for newcomers

help wanted

Extra attention is needed

invalid

This doesn't seem right

question

Further information is requested

wontfix

This will not be worked on

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

ai-review (AI Review) gb_barkley (Barkley Trimsworth) cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) gb_flea (Flea Flicker) flux (Flux CD) admin (Gitea Admin) gb_lint (Lint Roller) renovate (Mend Renovate) gb_pawla (Pawla Abdul) gb_scrubs (Scrubs McBarkley) gb_shedward (Shedward Scissorhands) gb_dogfather (The Dogfather)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: groombook/web#27