Skip to content

fix(docs): correct internal links by setting Astro site to domain#31

Merged
kattsushi merged 1 commit into
masterfrom
fix/docs-site-domain
Feb 7, 2026
Merged

fix(docs): correct internal links by setting Astro site to domain#31
kattsushi merged 1 commit into
masterfrom
fix/docs-site-domain

Conversation

@kattsushi

@kattsushi kattsushi commented Feb 7, 2026

Copy link
Copy Markdown
Contributor

Set site to https://devx-op.github.io and base to /effectify/ so all internal links resolve under https://devx-op.github.io/effectify/. This fixes 404s when navigating to pages like /react/getting-started/.

Summary by CodeRabbit

  • Chores
    • Updated documentation site URL to reflect new hosting location.

@coderabbitai

coderabbitai Bot commented Feb 7, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

The Astro configuration site URL was updated from https://devx-op.github.io/effectify/ to https://devx-op.github.io. All other configuration settings remain unchanged.

Changes

Cohort / File(s) Summary
Astro Configuration
apps/docs/astro.config.ts
Updated site property to change the base domain URL, removing the /effectify/ path suffix.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • devx-op/effectify#30: Also modifies the same Astro config file with related site and base configuration updates.

Poem

🐰 A path was shortened, clean and neat,
The URL now stands complete,
From nested deep to root so bright,
The docs now sit at heights so right! ✨

🚥 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 accurately summarizes the main change: updating the Astro site configuration to fix internal link resolution by setting the site to the domain root.
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-site-domain

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 caf1088

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=lint --base=origin/master ... ✅ Succeeded <1s View ↗
nx affected --target=typecheck --base=origin/ma... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-07 16:06:14 UTC

@kattsushi kattsushi merged commit 1a7984b 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