Skip to content

docs: fixed 404 error for high-performance pooling link#2537

Merged
unatasha8 merged 2 commits into
masterfrom
una-404-fix
Apr 30, 2026
Merged

docs: fixed 404 error for high-performance pooling link#2537
unatasha8 merged 2 commits into
masterfrom
una-404-fix

Conversation

@unatasha8
Copy link
Copy Markdown
Contributor

@unatasha8 unatasha8 commented Apr 30, 2026

Related Issue or Design Document

Checklist

  • I have read the contributing guidelines and signed the CLA.
  • I have referenced an issue containing the design document if my change introduces a new feature.
  • I have read the security policy.
  • I confirm that this pull request does not address a security vulnerability.
    If this pull request addresses a security vulnerability,
    I confirm that I got approval (please contact security@ory.com) from the maintainers to push the changes.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added the necessary documentation within the code base (if appropriate).

Further comments

Summary by CodeRabbit

  • Documentation
    • Fixed documentation links in the self-hosted deployment section. Navigation references for high-performance pooling information have been updated to point to the correct centralized documentation pages. This ensures users are consistently and reliably directed to the appropriate resources across the documentation system, improving overall navigation and user experience.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 30, 2026

Warning

Rate limit exceeded

@unatasha8 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 44 minutes and 36 seconds before requesting another review.

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 9e8e1dd4-1a5e-40a6-a12e-d3b99a3bbd7e

📥 Commits

Reviewing files that changed from the base of the PR and between 2d4ef65 and e95d53c.

📒 Files selected for processing (1)
  • src/components/Shared/self-hosted/deployment.tsx
📝 Walkthrough

Walkthrough

Two occurrences of a "high-performance pooling" link within the deployment component have been updated from relative page-internal targets to a shared self-hosted documentation route, altering navigation paths across the documentation structure.

Changes

Cohort / File(s) Summary
Documentation Link Navigation
src/components/Shared/self-hosted/deployment.tsx
Updated two internal navigation links for "high-performance pooling" from page-relative targets (./oel/... and #high-performance-pooling) to a consolidated cross-page route (../self-hosted/oel/high-performance-pooling).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐰 Hop, hop, the links now redirect with grace,
From scattered paths to one unified place,
High-performance pooling found its true home,
Through self-hosted docs, no more to roam!

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ⚠️ Warning The pull request description is entirely empty except for the template structure. No big picture explanation, related issue link, or justification is provided. Fill in the description with: (1) explanation of the 404 error and the fix, (2) related issue link in #XXXX format, and (3) check the appropriate checklist items, especially the CLA and contributing guidelines acknowledgement.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: fixing a 404 error for the high-performance pooling link by updating link targets in the deployment component.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch una-404-fix

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
Review rate limit: 0/1 reviews remaining, refill in 44 minutes and 36 seconds.

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

Copy link
Copy Markdown
Collaborator

@jaygiang jaygiang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ✅ Thanks!

@unatasha8 unatasha8 merged commit 05f1e68 into master Apr 30, 2026
11 checks passed
@unatasha8 unatasha8 deleted the una-404-fix branch April 30, 2026 04:50
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.

3 participants