Repository navigation
docs: make the root landing page route by job - #2393
Conversation
| --- | ||
|
|
||
| ## AI Agent Infrastructure | ||
| ## What do you want to do? |
There was a problem hiding this comment.
AGENTS.md §3 bans question headings outside FAQs and wants one or two sentences of prose before the first ## (Index type: one paragraph → CardGroup). Suggest a short intro line and e.g. ## Start here.
There was a problem hiding this comment.
Alternatively we can change the AGENTS.md. It is not obvious to me that these rules are beneficial.
| <Card title="ZK Identity" icon="passport" href="/build/build-with-self"> | ||
| Prove users’ humanity and unique identity without compromising privacy via Self Protocol. | ||
| <Card title="Deploy your first app" icon="bolt" href="/build/quickstart"> | ||
| Scaffold an app and deploy it to Celo Sepolia. |
There was a problem hiding this comment.
/build/quickstart scaffolds the app and runs it locally; deploying to Celo Sepolia only applies to the optional contracts workspace. Suggest "Scaffold an app and run it locally." Same for the Quickstart card at line 80.
palango
left a comment
There was a problem hiding this comment.
The routing works. I checked every internal link and they all land on pages in nav, and the card facts match the wording in AGENTS.md section 7. Two small fixes before merge. #2394 merged this morning, so the Contribute card's /contribute-to-celo now only works through a redirect; please point it at /contribute. And the Quickstart wording karlb flagged on line 27 also applies to the card title on line 26 and the accordion card on line 80: the quickstart scaffolds an app and runs it locally, and only the optional contracts workspace deploys to Celo Sepolia.
Optional, fine for a later pass: with the new top sections, the "Understanding Celo" and "Build with Celo" accordions mostly repeat the same links (/learn, /build, /build/quickstart, /learn/network/overview).
| <Card title="Operate" icon="server" href="/operate"> | ||
| Run nodes, follow network notices and read the protocol specification. | ||
| </Card> | ||
| <Card title="Contribute" icon="people-group" href="/contribute-to-celo"> |
There was a problem hiding this comment.
#2394 has merged, so contribute-to-celo/ no longer exists and this link only works through the /contribute-to-celo redirect. Please point it at /contribute.
| </Card> | ||
| <Card title="ZK Identity" icon="passport" href="/build/build-with-self"> | ||
| Prove users’ humanity and unique identity without compromising privacy via Self Protocol. | ||
| <Card title="Deploy your first app" icon="bolt" href="/build/quickstart"> |
There was a problem hiding this comment.
Same point as karlb's comment on the next line: the quickstart scaffolds an app and runs it locally, so "Deploy your first app" promises more than the page does. Maybe "Start your first app".
Replace the card stack on learn/celo with jobs-to-be-done cards and a four-tab map, drop the inflated description, and point the quickstart cards at the quickstart page. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Point the Contribute card at /contribute, say the quickstart scaffolds and runs an app locally, and open the page with a sentence and a non-question heading. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
a492873 to
b7e921c
Compare
|
Rebased onto main (#2394 and #2400 are in). Pushed fixes: the Contribute card points at |
palango
left a comment
There was a problem hiding this comment.
Thanks, both fixes are in. The Contribute card now points at /contribute, and the quickstart cards at the top and in the accordion say "run it locally", which matches what the quickstart does. The intro sentence and "Start here" also cover karlb's heading comment. I checked every internal link on the page again against docs.json, and they all land on pages in nav without going through a redirect. Fine to leave the accordion overlap for a follow-up. One small thing before merging: the PR body still mentions the "What do you want to do?" cards and "deploy a first app", so please update it if the squash commit uses it.
|
Updated the PR body to match the page: it now describes the intro and "Start here" section, the "run it locally" Quickstart wording and the |
Part of #2261 (first of three PRs; the Contribute rename and the AI resources page follow separately).
The root page opened with a single "Start Building" card and a stack of topic lists. It now leads with the jobs a first-time visitor comes to do, then maps the four tabs.
Changes
/build/quickstartdoes./contribute.descriptionno longer says "leading" (AGENTS.md section 4)./build/quickstart; the "Integrate" card is now labelled "Network information", which is where it links.learn/index.mdxstays Learn's overview and is not touched. The "Understanding Celo" and "Build with Celo" accordions still repeat some of the new links; trimming them is a follow-up.Verification
npx mint broken-links: no broken links found.🤖 Generated with Claude Code