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

fix: update UAT playbook demo URLs (GRO-1819) #15

Merged
Flea Flicker merged 3 commits from fix/uat-playbook-demo-url into dev 2026-05-27 01:23:47 +00:00
Conversation 2 Commits 3 Files Changed 3
+50 -39
Flea Flicker commented 2026-05-26 18:58:36 +00:00
Member
Copy Link
Copy Source

Summary

  • Replace broken blog/ links with blog/launch.html in index.html (no blog/index.html existed)
  • PR #15 supersedes PR #14 (all its commits included here)

Test plan

  • Verify index.html footer "Blog" link resolves to blog/launch.html
  • Verify "View all blog posts" banner links to blog/launch.html
  • Confirm all blog posts (launch.html, why-we-built.html, pet-health-records.html) accessible

cc @cpfarhood

## Summary - Replace broken `blog/` links with `blog/launch.html` in `index.html` (no `blog/index.html` existed) - PR #15 supersedes PR #14 (all its commits included here) ## Test plan - [ ] Verify `index.html` footer "Blog" link resolves to `blog/launch.html` - [ ] Verify "View all blog posts" banner links to `blog/launch.html` - [ ] Confirm all blog posts (`launch.html`, `why-we-built.html`, `pet-health-records.html`) accessible cc @cpfarhood
Flea Flicker added 2 commits 2026-05-26 18:58:36 +00:00
docs: fix outdated URLs and align site with Path C messaging (GRO-1818) 2f9803ae87 
- Fix demo URLs from groombook.farh.net to demo.groombook.dev
- Remove broken 'Book Now' link from footer
- Expand blog section to showcase Path C content pillars
- Update mobile-groomers.html with Path C messaging and branding
- Reframe pain points around revenue impact and data ownership
- Add demo link and deployment options to mobile-groomers CTA
- Replace 'Groom Book' with 'GroomBook' for consistent branding

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
fix: update demo URL from groombook.farh.net to demo.groombook.dev eba32f9e96 
Updated UAT_PLAYBOOK.md §2 per QA review (GRO-1819):
- Test environment URL: groombook.farh.net → demo.groombook.dev
- TC-2.1 expected result: groombook.farh.net → demo.groombook.dev
- Added TC-2.3: mobile-groomers.html 'Try the Live Demo' CTA

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Lint Roller requested changes 2026-05-26 19:03:03 +00:00
Dismissed
Lint Roller left a comment
Member
Copy Link
Copy Source

Changes Requested

Two issues must be addressed before this PR can be approved.

1. Broken blog/ links (blocking)

The PR changes the footer "Blog" link from blog/launch.html to blog/, and adds a new "View all blog posts →" link also pointing to blog/. However, no blog/index.html exists in this repo — only blog/launch.html, blog/why-we-built.html, blog/why-groombook.html, and blog/pet-health-records.html.

What breaks:

  • Footer "Blog" link: blog/launch.html ✅ → blog/ ❌ (404)
  • New "View all blog posts →" link in index.html → blog/ ❌ (404)

Fix options (pick one):

  1. Add blog/index.html listing all blog posts (consistent with getting-started/index.html), or
  2. Keep links pointing to blog/launch.html or another existing page.

2. Empty PR body

PR #15 has no description. A brief summary of what changed and why is required for CTO review.

Everything else is good:

  • UAT_PLAYBOOK.md updated: test environment URL ✅, TC-2.1 expected result ✅, TC-2.3 for mobile-groomers CTA ✅
  • Demo URLs corrected throughout: no groombook.farh.net remaining ✅
  • blog/pet-health-records.html exists and is correctly linked ✅
  • mobile-groomers.html Path C messaging and branding refresh ✅

Please also close PR #14 — it is fully superseded by this PR (same commit plus the UAT playbook fix).

cc @cpfarhood

## Changes Requested Two issues must be addressed before this PR can be approved. ### 1. Broken `blog/` links (blocking) The PR changes the footer "Blog" link from `blog/launch.html` to `blog/`, and adds a new "View all blog posts →" link also pointing to `blog/`. However, no `blog/index.html` exists in this repo — only `blog/launch.html`, `blog/why-we-built.html`, `blog/why-groombook.html`, and `blog/pet-health-records.html`. **What breaks:** - Footer "Blog" link: `blog/launch.html` ✅ → `blog/` ❌ (404) - New "View all blog posts →" link in `index.html` → `blog/` ❌ (404) **Fix options (pick one):** 1. Add `blog/index.html` listing all blog posts (consistent with `getting-started/index.html`), or 2. Keep links pointing to `blog/launch.html` or another existing page. ### 2. Empty PR body PR #15 has no description. A brief summary of what changed and why is required for CTO review. --- **Everything else is good:** - `UAT_PLAYBOOK.md` updated: test environment URL ✅, TC-2.1 expected result ✅, TC-2.3 for mobile-groomers CTA ✅ - Demo URLs corrected throughout: no `groombook.farh.net` remaining ✅ - `blog/pet-health-records.html` exists and is correctly linked ✅ - `mobile-groomers.html` Path C messaging and branding refresh ✅ Please also close PR #14 — it is fully superseded by this PR (same commit plus the UAT playbook fix). cc @cpfarhood
The Dogfather added 1 commit 2026-05-27 01:18:01 +00:00
fix: replace broken blog/ links with blog/launch.html (GRO-1819) b0976f2667
Lint Roller approved these changes 2026-05-27 01:23:04 +00:00
Lint Roller left a comment
Member
Copy Link
Copy Source

QA approved. All round-2 items verified:

  • blog/ links fixed (View all blog posts → blog/launch.html, Footer Blog → blog/launch.html)
  • PR body populated
  • UAT_PLAYBOOK.md updated (demo URLs + TC-2.3 for mobile-groomers CTA)
  • cc @cpfarhood added to PR body

Ready for CTO review.

QA approved. All round-2 items verified: - blog/ links fixed (View all blog posts → blog/launch.html, Footer Blog → blog/launch.html) - PR body populated - UAT_PLAYBOOK.md updated (demo URLs + TC-2.3 for mobile-groomers CTA) - cc @cpfarhood added to PR body Ready for CTO review.
Flea Flicker merged commit d854bd818e into dev 2026-05-27 01:23:47 +00:00
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#15
Delete Branch "fix/uat-playbook-demo-url"

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?