-
Notifications
You must be signed in to change notification settings - Fork 41
Fix broken link to labels.html in print.html documentation #414
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
WalkthroughA single documentation link in doc/book/print.html was updated to point to labels.html instead of mammothon/./labels.html. No code, runtime logic, or public APIs were changed. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Nitpick comments (1)
doc/book/print.html (1)
540-541: Unrelated but nearby: broken link to “architechture.html”.There’s no “architechture.html” page in this book; elsewhere (Line 518) the page is “architecture.html”. This link will 404 in print and web builds.
Apply:
-<li>Starting a <a href="./architechture.html">local prism devnet</a></li> +<li>Starting a <a href="./architecture.html">local prism devnet</a></li>
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
doc/book/print.html(1 hunks)
| <ul> | ||
| <li><a href="https://www.reclaimprotocol.org">Reclaim Protocol</a></li> | ||
| <li><a href="mammothon/./labels.html">Prism Account Sources</a></li> | ||
| <li><a href="labels.html">Prism Account Sources</a></li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
Fix looks correct; relative link now matches other sidebar/menu links.
Switching to "labels.html" resolves the 404 and is consistent with other references in this file (e.g., the sidebar at Line 91 already uses labels.html). Nice catch.
Optionally run this quick check to ensure no other stale variants remain:
🏁 Script executed:
#!/bin/bash
# Find any lingering broken or inconsistent references
rg -nP 'mammothon/\.?/labels\.html|mammothon/labels\.html|architechture\.html' -C2 doc/Length of output: 455
Fix approved with one additional correction needed
Aside from updating the Prism Account Sources link, there’s a typo in the “architecture” link that will result in a 404. Please correct it as follows:
• File: doc/book/print.html, Line 540
Current:
<li>Starting a <a href="./architechture.html">local prism devnet</a></li>Updated:
- <li>Starting a <a href="./architechture.html">local prism devnet</a></li>
+ <li>Starting a <a href="./architecture.html">local prism devnet</a></li>With that change, all sidebar/menu links will be consistent and free of broken references.
🤖 Prompt for AI Agents
In doc/book/print.html around line 540, the sidebar/menu link has a typo in the
href ("./architechture.html") that will cause a 404; update the href to the
correct filename "./architecture.html" so the list item reads the same but
points to the correct architecture page, ensuring all sidebar links are
consistent and not broken.
|
These actually must be modified in the .md files, the .html files are compiled from the md |
I can't find this link in .md files |
Fix broken link to labels.html that was causing 404 errors
Summary by CodeRabbit