ReviewOS

also looking at this

chris/site

fix(goals): link the training note at the shared plan, not the homepage

#2
Open chris wants to merge fix/goals-share-link into main
1 file +1 -1
8 days ago

The goals page says the training behind the running half is "the same app I build for everyone else, pointed at my own weeks" — and then linked the HQ.training marketing homepage. With the coming-soon gate up that lands on a holding page, so the one sentence about the actual training pointed at nothing about it.

It now links https://hq.training/share/chris, the read-only shared page that shows those weeks: recent workouts, daily health metrics, goals, and the plan ahead. No login, noindex, and revocable from Profile → Public progress link at any time.

The slug is reserved rather than minted, so it survives the demo dataset being deleted and recreated on every deploy — the link will not rot into a 404.

The HQ.training entry in the work list above is deliberately unchanged: that goal is about building the product, so the product homepage remains the right destination for it.

Verified: destination returns 200 in production; page renders locally with both links resolving as intended (https://hq.training/ for the work goal, https://hq.training/share/chris for the training note); pickier clean on the changed file.

One line changed.

🤖 Generated with Claude Code

1 changed file on the files tab, with 0 review threads.

Sign in to comment on this pull request.