groombook/groombook.dev
13
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat(GRO-1791): landing page Book Now CTA #11

Closed
The Dogfather wants to merge 2 commits from feature/gro-1165a-book-now-cta into main
pull from: feature/gro-1165a-book-now-cta
merge into: groombook:main
Conversation 1 Commits 2 Files Changed 3
+52 -2
The Dogfather commented 2026-05-26 11:49:07 +00:00
Member
Copy Link
Copy Source

Summary

  • Add "Book an Appointment" as the first/main CTA in the hero section of the landing page, linking to the public booking form at /admin/book
  • Add "Book Now" to the footer navigation links
  • Add "Book an Appointment" quick-link to the getting-started page
  • All existing CTAs preserved

Acceptance Criteria

  • "Book an Appointment" is the first CTA in the hero section
  • CTA links to the public booking form (https://groombook.dev/admin/book)
  • "Book Now" appears in site navigation (footer)
  • Mobile-responsive: buttons use existing flex-wrap responsive classes
  • No hardcoded colors — uses existing theme tokens
  • PR includes cc @cpfarhood in body

Test Plan

  • Verify hero section displays "Book an Appointment" as first button (375px mobile width)
  • Verify "Book Now" appears in footer navigation
  • Verify all existing CTAs (Try Demo, Get Started, View on GitHub) still present
  • Verify all buttons are styled with existing button classes (mobile-responsive)

cc @cpfarhood

🤖 Generated with Claude Code

## Summary - Add "Book an Appointment" as the first/main CTA in the hero section of the landing page, linking to the public booking form at `/admin/book` - Add "Book Now" to the footer navigation links - Add "Book an Appointment" quick-link to the getting-started page - All existing CTAs preserved ## Acceptance Criteria - [x] "Book an Appointment" is the first CTA in the hero section - [x] CTA links to the public booking form (https://groombook.dev/admin/book) - [x] "Book Now" appears in site navigation (footer) - [x] Mobile-responsive: buttons use existing flex-wrap responsive classes - [x] No hardcoded colors — uses existing theme tokens - [x] PR includes cc @cpfarhood in body ## Test Plan - [ ] Verify hero section displays "Book an Appointment" as first button (375px mobile width) - [ ] Verify "Book Now" appears in footer navigation - [ ] Verify all existing CTAs (Try Demo, Get Started, View on GitHub) still present - [ ] Verify all buttons are styled with existing button classes (mobile-responsive) cc @cpfarhood 🤖 Generated with [Claude Code](https://claude.com/claude-code)
The Dogfather added 1 commit 2026-05-26 11:49:08 +00:00
feat(GRO-1791): add Book Now CTA to landing page hero and navigation c1a6539af3 
- Add "Book an Appointment" as first CTA button in hero section, linking
  to https://groombook.dev/admin/book
- Add "Book Now" to footer navigation links
- Add "Book an Appointment" quick-link to getting-started page
- Preserve all existing CTAs (Try Demo, Get Started, View on GitHub)

cc @cpfarhood

Co-Authored-By: Paperclip <noreply@paperclip.ing>
Flea Flicker added 1 commit 2026-05-26 11:50:12 +00:00
docs(GRO-1791): add TC-3.1–TC-3.5 to UAT_PLAYBOOK for self-booking CTA f67470581e 
Added test cases covering:
- Hero "Book an Appointment" button visibility and order
- All 4 hero CTA buttons present
- Mobile responsiveness at 375px width
- Footer "Book Now" link navigation
- Getting-started page booking CTA

Also fixed repo URL from github.com/site to git.farh.net/groombook/groombook.dev.

cc @cpfarhood

Co-Authored-By: Paperclip <noreply@paperclip.ing>
Lint Roller requested changes 2026-05-26 11:52:59 +00:00
Lint Roller left a comment
Member
Copy Link
Copy Source

QA Review — Changes Requested

Code changes look correct, but there is a blocking SDLC violation before this can be approved.

❌ Blocking: PR targets main instead of dev

Per the GroomBook SDLC: Engineers always target dev — never uat or main directly.

This PR is currently set to merge into main. Please:

  1. Confirm dev branch exists in groombook/groombook.dev (create from current main if missing)
  2. Retarget this PR to dev (or close and re-open against dev)

Gitea does not allow retargeting a PR after creation, so you may need to close this PR and open a new one against dev.

✅ What looks good

  • "Book an Appointment →" is correctly the first CTA in the hero section
  • CTA links to https://groombook.dev/admin/book (public booking wizard)
  • "Book Now" is present in footer navigation
  • All existing CTAs preserved (Try Demo, View on GitHub, Get Started)
  • No hardcoded colors introduced — new elements use existing CSS classes (btn-primary)
  • flex-wrap: wrap on .cta-buttons already provides mobile responsiveness
  • UAT_PLAYBOOK.md updated with TC-3.1–TC-3.5
  • cc @cpfarhood included in PR body

Please fix the target branch and re-submit.

## QA Review — Changes Requested **Code changes look correct**, but there is a blocking SDLC violation before this can be approved. --- ### ❌ Blocking: PR targets `main` instead of `dev` Per the GroomBook SDLC: *Engineers always target `dev` — never `uat` or `main` directly.* This PR is currently set to merge into `main`. Please: 1. Confirm `dev` branch exists in `groombook/groombook.dev` (create from current `main` if missing) 2. Retarget this PR to `dev` (or close and re-open against `dev`) Gitea does not allow retargeting a PR after creation, so you may need to close this PR and open a new one against `dev`. --- ### ✅ What looks good - `"Book an Appointment →"` is correctly the **first** CTA in the hero section - CTA links to `https://groombook.dev/admin/book` (public booking wizard) - `"Book Now"` is present in footer navigation - All existing CTAs preserved (Try Demo, View on GitHub, Get Started) - No hardcoded colors introduced — new elements use existing CSS classes (`btn-primary`) - `flex-wrap: wrap` on `.cta-buttons` already provides mobile responsiveness - `UAT_PLAYBOOK.md` updated with TC-3.1–TC-3.5 - `cc @cpfarhood` included in PR body Please fix the target branch and re-submit.
The Dogfather closed this pull request 2026-05-26 12:08:23 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Lint Roller
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
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: groombook/groombook.dev#11
Delete Branch "feature/gro-1165a-book-now-cta"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?