Skip to content

fix the page , now the link is working#508

Closed
MYMISERABLEMIND17 wants to merge 1 commit intoCircuitVerse:masterfrom
MYMISERABLEMIND17:master
Closed

fix the page , now the link is working#508
MYMISERABLEMIND17 wants to merge 1 commit intoCircuitVerse:masterfrom
MYMISERABLEMIND17:master

Conversation

@MYMISERABLEMIND17
Copy link
Copy Markdown

@MYMISERABLEMIND17 MYMISERABLEMIND17 commented Jan 26, 2026

The "Edit this page" link was generating an incorrect GitHub path
with a duplicated docs/docs segment.

This PR updates the Docusaurus editUrl configuration to correctly
map documentation pages to their editable GitHub source files.

Tested locally on multiple documentation pages.
Screenshot from 2026-01-26 19-58-55

basically in introduction page " Edit this page link is now working "

merge this pull request

Summary by CodeRabbit

  • Chores
    • Updated documentation editing configuration URL path.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 26, 2026

Walkthrough

The docusaurus.config.js configuration file has been modified to update the editUrl field within the docs object. The edit URL path has been changed from https://github.com/CircuitVerse/CircuitVerseDocs/edit/master/docs/ to https://github.com/CircuitVerse/CircuitVerseDocs/edit/master/. This adjustment removes the /docs/ segment from the documentation edit URL. Additional blank lines were introduced as non-functional formatting changes. The modification involved 3 additions and 1 deletion across the affected lines.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title refers to fixing a page link and states the link is now working, which aligns with the actual change (fixing the editUrl configuration). However, the phrasing is casual and somewhat vague—it doesn't explicitly identify what was fixed (the editUrl path duplication issue) or mention the specific technical component being modified.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@MYMISERABLEMIND17
Copy link
Copy Markdown
Author

MYMISERABLEMIND17 commented Jan 26, 2026 via email

@Nihal4777
Copy link
Copy Markdown
Member

Thanks @MYMISERABLEMIND17, Closing this as it is fixed in #501.

@Nihal4777 Nihal4777 closed this Jan 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants