Conversation
Ship the SQLite-ledger H1, two-beat subhead (local DB + Plaid/Copilot comparison), and two first-class CTAs (14-day trial download + live demo to /overview). Update SITE.tagline/description and OG metadata so the old slogan does not linger in social cards. Co-authored-by: Kevin George <kGeee@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Attach current-state still of new H1 + dual CTAs under .pr-screenshots. Co-authored-by: Kevin George <kGeee@users.noreply.github.com>
Screenshots (current state)Homepage hero with new H1 + dual CTAs (
SHA: will update after push — see latest commit on |
|
Screenshot SHA:
|
kGeee
left a comment
kGeee
left a comment
There was a problem hiding this comment.
Verdict: COMMENT (hold) — code-clear vs claims; would APPROVE if undrafted. Never APPROVE while draft.
Head SHA: c461d7787f25bc138cbd4168be822beefb2a9c50
CI / checks: green
web— successVercel Preview Comments— success- Vercel deployment — success (Ready)
Screenshot gate: PASS — Kevin posted homepage-hero.png for this SHA (PR body + comments #5517277875 / #5517280264).
Secrets: clear (manual scan of diff; GHAS run_secret_scanning unavailable on this repo). Pre-existing Polar CHECKOUT_URL in site.ts is unchanged and is a public checkout link, not a secret.
Code vs claims
Matches the PR brief and stays in scope (+21/−19 across marketing copy/metadata + screenshot):
- Eyebrow → Private personal finance for Apple Silicon
- H1 via
SITE.tagline→ Your ledger is a SQLite file on this Mac. - Two-beat subhead:
SITE.description+ Plaid BYO / $29 once vs Copilot Money Google Cloud $95/year (comparison only in hero JSX, not OG) - CTAs:
DownloadLink label="Download the 14-day trial"(same DMG viadownloadHref()/directDmgUrl) + gatedTry the live demo→DEMO_HREF(/overview) viademoEnabled() - Tertiary See the app (
#screens) removed from hero SITE.tagline/SITE.descriptionupdated; page/layout OG+Twitter titles wired toSITE.tagline; OG description stays first beat only- Untouched as claimed: privacy band, onboarding Plaid steps, no
/privacy|/terms|/demolinks, no Windows/Electron/NSIS/licensing
DownloadLink already accepts optional label (marketing-shell.tsx); no href change.
Blockers
- Remains draft — do not APPROVE; PR Merger stays OFF until undraft.
- No other merge blockers from this review (mergeable_state clean, checks green, screenshots current).
Soft nits (non-blocking)
- OG title vs document title:
page.tsxdocument title isbudgetr — Private personal finance for Apple Silicon, while OG/Twitter titles are bareSITE.tagline(nobudgetr —prefix). Intentional for social cards is fine; just inconsistent branding in the tab title vs share card. - Demo CTA weight: live-demo link dropped brass highlight for
border-line/ paper text (closer to the old tertiary). Still readable as a second CTA next to primary Download; Design may want brass parity if both are meant to be equally first-class. - alt.txt: still no trailing newline (pre-existing style; unchanged behavior).
PR Merger: OFF (draft).
2 participants
Replace the live budgetr.dev homepage hero copy for the MARKETING_ONLY deploy (
/landing,/overviewlive demo).Copy (verbatim)
/overview(still gated bydemoEnabled()/DEMO_DB)Metadata
SITE.tagline/SITE.descriptionupdated so OG/Twitter and alt text no longer carry the old sloganUnchanged
/overviewdemo banner text and sample-data disclaimers/privacy,/terms, or/demolinksScreenshots
Verified locally
MARKETING_ONLY=1 DEMO_DB=1homepage renders new eyebrow/H1/two-beat subhead and two first-class CTAs/overviewdemo banner text unchangedDo not merge until reviewed. Stays draft.
Head:
c461d7787f25bc138cbd4168be822beefb2a9c50