Conversation
Add ordered guides from saving through investing, charts, analysis, and options, ending on the SQLite ledger product page. Pages 1–10 link Next only; page 11 carries the frozen demo closer. Guides nav points at /learn without touching Download or the hero. Co-authored-by: Kevin George <kGeee@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
kGeee
left a comment
There was a problem hiding this comment.
Verdict: COMMENT (hold merge) — head 83001f7
11-article knowledge path vs Marketing lock. CI green (web). Still a draft — not an approve. Do not merge. #40 stays HOLD. #35/#36/#37 untouched.
Lock
| Item | Status |
|---|---|
11 articles, curriculum order, locked slugs in web/lib/learn.ts |
Pass |
| Index H1 “Learn, in order”; ordered title list, no screenshots | Pass |
/learn/[slug] + generateStaticParams |
Pass |
| Pages 1–10 Next only (no product pitch) | Pass |
Page 11 only: SQLite closer + /overview demo |
Pass |
| Guides in header + footer; Download / hero untouched | Pass |
| Investing pages keep “education, not advice” | Pass |
| Only labeled $4,000 / $12,000 example; no Copilot $95 / farm hooks | Pass |
(marketing) / MARKETING_ONLY |
Pass |
| No secrets | Pass |
Soft (non-blocking)
- Page 11 closer renders two links to the same href (
DEMO_HREF);LEARN_PRODUCT_CLOSER.hrefis unused. - Index eyebrow says “Guides” while article crumbs say “Learn” — both map to
/learn.
Code clear once undrafted. @pr Merger stay off until then. I will not merge.
Replace curriculum copy with the frozen v2 paragraphs. Drop LEARN_PRODUCT_CLOSER so page 11 renders the SQLite demo closer once. Co-authored-by: Kevin George <kGeee@users.noreply.github.com>
kGeee
left a comment
There was a problem hiding this comment.
Verdict: COMMENT (hold merge) — head 7830ec0
v2 paste is in. CI green (web + Vercel). Still a draft — not an approve. Do not merge. #40 stays HOLD. #35/#36/#37 untouched.
Lock vs 83001f7
| Item | Status |
|---|---|
Frozen v2 paragraphs in web/lib/learn.ts (11 slugs, labeled examples) |
Pass |
LEARN_PRODUCT_CLOSER dropped; page 11 closer inlined once |
Pass |
| Index H1 “Learn, in order”; ordered titles, no screenshots | Pass |
/learn/[slug] + generateStaticParams |
Pass |
| Pages 1–10 Next only (no product CTA) | Pass |
Page 11 only: SQLite closer + /overview demo |
Pass |
| Guides in header + footer; Download / hero untouched | Pass |
| Investing/options keep “education, not advice”; 3D IV/smile/1256 out | Pass |
| Plaid BYO-keys + $29 only on page 11 | Pass |
| Labeled example $ totals; no Copilot $95 / farm hooks | Pass |
(marketing) / MARKETING_ONLY |
Pass |
| No secrets | Pass |
Soft (non-blocking)
- Page 11 still has two
Links toDEMO_HREF(“Try the live demo” +/overview). Closer copy is no longer doubled. - Index eyebrow says “Guides” while article crumbs say “Learn” — both map to
/learn.
Code clear once undrafted. @pr Merger stay off until then. I will not merge.
| <Link | ||
| href={DEMO_HREF} | ||
| className="inline-flex items-center gap-1.5 font-medium text-[var(--brass)] underline decoration-[var(--brass-dim)] underline-offset-4 transition hover:decoration-[var(--brass)]" | ||
| > | ||
| Try the live demo (sample data) | ||
| <ArrowRight size={15} /> | ||
| </Link> | ||
| <span className="text-[var(--faint)]">: </span> | ||
| <Link | ||
| href={DEMO_HREF} | ||
| className="font-medium text-[var(--brass)] underline decoration-[var(--brass-dim)] underline-offset-4 transition hover:decoration-[var(--brass)]" | ||
| > | ||
| /overview | ||
| </Link> | ||
| </p> | ||
| </section> | ||
| ) : next ? ( |
There was a problem hiding this comment.
Still two Links to the same DEMO_HREF (“Try the live demo (sample data)” and /overview). Dropping LEARN_PRODUCT_CLOSER fixed the doubled closer copy; this is the leftover twin CTA. Soft — one link is enough.
Attach index + saving article stills under .pr-screenshots for draft PR review. Co-authored-by: Kevin George <kGeee@users.noreply.github.com>
kGeee
left a comment
There was a problem hiding this comment.
Verdict: COMMENT (hold merge) — head 8f902f3
Screenshot gate cleared: current-state index + saving article attached. CI green (web + Vercel). Still a draft — not an approve. Do not merge. #40 stays HOLD. #35/#36/#37 untouched.
Lock vs 7830ec0
| Item | Status |
|---|---|
Frozen v2 paragraphs in web/lib/learn.ts (11 slugs, labeled examples) |
Pass |
| Index H1 “Learn, in order”; ordered titles, no screenshots on page | Pass |
/learn/[slug] + generateStaticParams |
Pass |
| Pages 1–10 Next only (no product CTA) | Pass |
Page 11 only: SQLite closer + /overview demo |
Pass |
| Guides in header + footer; Download / hero untouched | Pass |
| Investing/options keep “education, not advice”; 3D IV/smile/1256 out | Pass |
| Plaid BYO-keys + $29 only on page 11 | Pass |
| Labeled example $ totals; no Copilot $95 / farm hooks | Pass |
(marketing) / MARKETING_ONLY |
Pass |
| Current screenshots (index + saving) | Pass |
| No secrets | Pass |
Soft (non-blocking)
- Page 11 still has two
Links toDEMO_HREF(“Try the live demo” +/overview). - Index eyebrow says “Guides” while article crumbs say “Learn” — both map to
/learn.
Code clear once undrafted. @pr Merger stay off until then. I will not merge.
| return ( | ||
| <main className="mx-auto max-w-3xl px-5 py-16 sm:px-8"> | ||
| <div className="max-w-2xl"> | ||
| <p className="eyebrow">Guides</p> |
There was a problem hiding this comment.
Soft (non-blocking): index eyebrow is “Guides” while article crumbs say “Learn”. Both go to /learn — fine, just a naming mismatch.
| </Link> | ||
| </p> | ||
| </section> | ||
| ) : next ? ( |
There was a problem hiding this comment.
Soft (non-blocking): two Links both point at DEMO_HREF (“Try the live demo” and /overview). Closer copy is single; the double link is intentional labeling, but worth a quick eye if you want one CTA only.




Ship
budgetr.dev/learnas an 11-article knowledge path (not the slideshow farm). Branched frommain; leaves #35 / #36 / #37 / #40 alone. Stays draft — do not merge.What
web/lib/learn.ts— 11 articles, locked slugs, v2 verbatim body copy/learn: H1 “Learn, in order”, ordered title list (no screenshots)/learn/[slug]viagenerateStaticParams/overviewLink (no doubledLEARN_PRODUCT_CLOSER)Constraints
(marketing)layout; Fraunces + brass accent; no new brandScreenshots
Test plan
/learn+ all 11 slugs 200/learn/overview(no double closer)Head:
8f902f37f0b2e9de7781664974a5226e7c4aef79