Repository navigation
fix(links): rename "run a node" CTA to "read the docs" - #44
alpergundogdu wants to merge 1 commit into
Conversation
Both CTAs already pointed at the docs URL; aligning the label and dropping the now-unused `runNode` link entry.
Deploying website with
|
| Latest commit: |
3dfd078
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://0b3edc8c.website-70y.pages.dev |
| Branch Preview URL: | https://claude-update-link-text-ymiq.website-70y.pages.dev |
There was a problem hiding this comment.
Pull request overview
Updates the marketing site CTA wording to better match the destination (documentation) and removes an unused link constant from the shared links registry.
Changes:
- Renamed Hero (
#intro) CTA text from “run a node” to “read the docs”. - Renamed Close/Contact (
#contact) CTA text from “run a node” to “read the docs”. - Repointed both CTAs to
links.docsand removed the unusedlinks.runNodeentry.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
lib/links.ts |
Removes the now-unused runNode link key, keeping docs as the canonical docs URL. |
components/site/Hero.tsx |
Updates the Hero CTA label and switches its href to links.docs. |
components/site/Close.tsx |
Updates the Close/Contact CTA label and switches its href to links.docs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request replaces "run a node" links with "read the docs" links in the Hero and Close components and removes the runNode key from the central links configuration. Feedback highlights that removing this key is a breaking change and suggests verifying that other components, such as Comparison, Method, or Faq, do not still rely on it.
|
Closing this PR as the issue has been addressed in #41. |
Summary
hreffromlinks.runNodetolinks.docsand drop the now-unusedrunNodeentry fromlib/links.ts.Test plan
pnpm lintclean#intro) and Contact (#contact) CTAs render as "read the docs →" and link tohttps://docs.decdn.org/overview/introductionGenerated by Claude Code