Skip to content

fix(docs): relative internal links under /effectify/ base - #32

Merged
kattsushi merged 1 commit into
masterfrom
fix/docs-relative-links
Feb 7, 2026
Merged

kattsushi merged 1 commit into
masterfrom
fix/docs-relative-links

Conversation

@kattsushi

@kattsushi kattsushi commented Feb 7, 2026 •

Copy link
Copy Markdown
Contributor
  • Use relative links in landing and navigation to ensure URLs resolve under /effectify/ on GitHub Pages\n- Keep site=https://devx-op.github.io and base=/effectify/ for correct asset and link generation\n- Fixes navigation to pages like /react/getting-started/

Summary by CodeRabbit

  • Documentation
    • Improved navigation structure across documentation sections
    • Expanded platform support information with new Universal packages section
    • Added Quick Start guides for React, SolidJS, and Backend platforms
    • Enhanced community section with new Discord community link for support

@coderabbitai

coderabbitai Bot commented Feb 7, 2026 •

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

Documentation site navigation and landing page links converted from absolute to relative paths. Sidebar configuration updated, and landing page restructured with new quick start sections and expanded platform support areas while standardizing quote syntax.

Changes

Cohort / File(s) Summary
Navigation Configuration
apps/docs/astro.config.ts
Converted sidebar navigation link targets from absolute paths (/react/, /solid/, etc.) to relative paths (react/, solid/, etc.) for ReactJs, SolidJs, Backend, and Universal sections.
Landing Page Content
apps/docs/src/content/docs/index.mdx
Replaced absolute link paths with relative routes throughout hero actions and card links. Standardized import quotes from single to double. Restructured Platform Support section with new Universal card, added Quick Start blocks for React, SolidJS, and Backend, and enhanced Community & Support section with new Discord and Discussions links.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Hops along the garden path,
No more absolute's aftermath,
Relative routes now lead the way,
Making links more local today!
Fresh quick starts and platforms bright, 🌟

🚥 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 'fix(docs): relative internal links under /effectify/ base' directly and clearly describes the main change—converting absolute paths to relative paths for links in documentation files under the GitHub Pages base directory.
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
  • Commit unit tests in branch fix/docs-relative-links

No actionable comments were generated in the recent review. 🎉


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

@nx-cloud

nx-cloud Bot commented Feb 7, 2026 •

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 44efecd

Command Status Duration Result
nx affected --target=test --base=87c40a1ce45893... ✅ Succeeded 10s View ↗
nx affected --target=build --base=87c40a1ce4589... ✅ Succeeded 16s View ↗
nx affected --target=typecheck --base=origin/ma... ✅ Succeeded <1s View ↗
nx affected --target=lint --base=origin/master ... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-07 21:23:41 UTC

@kattsushi
kattsushi merged commit 5d63524 into master Feb 7, 2026
7 checks passed
kattsushi added a commit that referenced this pull request Mar 15, 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.

1 participant