Skip to content

EPMRPP-114309 || Fix incorrect redirect for release documentation URL…#1107

Closed
maria-hambardzumian wants to merge 2 commits into
developfrom
fix/EPMRPP-114309-incorrect-redirect-v2
Closed

EPMRPP-114309 || Fix incorrect redirect for release documentation URL…#1107
maria-hambardzumian wants to merge 2 commits into
developfrom
fix/EPMRPP-114309-incorrect-redirect-v2

Conversation

@maria-hambardzumian
Copy link
Copy Markdown
Contributor

@maria-hambardzumian maria-hambardzumian commented Apr 29, 2026

…s v2

Summary by CodeRabbit

  • Chores
    • Improved documentation URL consistency and redirect handling.
    • Extended redirect coverage for legacy documentation paths and archived release versions.
    • Enhanced URL standardization across all documentation sections.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 29, 2026

Warning

Rate limit exceeded

@maria-hambardzumian has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 28 minutes and 46 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: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 581f0e8d-72a7-471e-b034-b570c8ba3d4a

📥 Commits

Reviewing files that changed from the base of the PR and between 26246de and 7a6e4b1.

📒 Files selected for processing (1)
  • docusaurus.config.js

Walkthrough

The docusaurus.config.js file's redirect configuration is updated to standardize destination URLs with trailing slashes, expand legacy source paths for SAML providers, SauceLabs, and ReportPortal API, modify fragment redirects to include forward slashes before anchors, and add extensive release/version redirect mappings with new and archived versions.

Changes

Cohort / File(s) Summary
Redirect Configuration Updates
docusaurus.config.js
Standardizes trailing slashes across destination URLs, broadens legacy from paths for provider routing (SAML, SauceLabs, ReportPortal), modifies fragment redirects to include / before # anchors, and adds comprehensive release/version redirect mappings covering current and archived releases.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 URLs hop with trailing slashes bright,
Fragments now fragment with forward slash might,
Legacy paths redirect with care,
Release versions everywhere!
Docs flow smooth without a snare. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: fixing incorrect redirects for release documentation URLs. It directly relates to the primary modification of updating redirect configurations in docusaurus.config.js.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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 fix/EPMRPP-114309-incorrect-redirect-v2

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 28 minutes and 46 seconds.

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

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
docusaurus.config.js (1)

588-806: Consider generating repetitive release redirects from data.

Lines 588–806 are highly repetitive. Building these entries from version arrays would reduce typo risk and future maintenance overhead.

♻️ Refactor sketch
+const canonicalReleaseVersions = [
+  '25.1.1', '25.0.4', '25.1.4', '25.0.5', '25.1.11',
+  '25.1', '24.1', '25.1.10', '24.2.2', '25.1.9',
+  // ...
+];
+
+const canonicalReleaseRedirects = canonicalReleaseVersions.map((v) => ({
+  to: `/releases/Version${v}/`,
+  from: `/releases/Version${v}`,
+}));
+
+const aliasedReleaseRedirects = [
+  {
+    to: '/releases/Version25.1.5/',
+    from: ['/releases/Release25.1.5', '/releases/Release25.1.5/', '/releases/Version25.1.5'],
+  },
+  {
+    to: '/releases/Version26.0.1/',
+    from: ['/releases/Release26.0.1', '/releases/Release26.0.1/', '/releases/Version26.0.1'],
+  },
+  // ...
+];
...
         redirects: [
           // existing non-release redirects...
-          { to: '/releases/Version25.1.1/', from: '/releases/Version25.1.1' },
-          { to: '/releases/Version25.1.5/', from: ['/releases/Release25.1.5', '/releases/Release25.1.5/', '/releases/Version25.1.5'] },
-          // ...many more...
+          ...canonicalReleaseRedirects,
+          ...aliasedReleaseRedirects,
         ],
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@docusaurus.config.js` around lines 588 - 806, The redirects list is manually
repeating many release entries; replace it by creating a structured array (e.g.,
const releases = [...]) and a helper (e.g., buildReleaseRedirects or
generateReleaseRedirects) that maps each version to the { to:
'/releases/VersionX/', from: [...] } objects and then spread the result into the
redirects array used in docusaurus config; update references in the config to
use that generated array (look for the redirects array and entries with keys
to/from in docusaurus.config.js) so future versions are added by updating the
releases array only.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@docusaurus.config.js`:
- Around line 588-806: The redirects list is manually repeating many release
entries; replace it by creating a structured array (e.g., const releases =
[...]) and a helper (e.g., buildReleaseRedirects or generateReleaseRedirects)
that maps each version to the { to: '/releases/VersionX/', from: [...] } objects
and then spread the result into the redirects array used in docusaurus config;
update references in the config to use that generated array (look for the
redirects array and entries with keys to/from in docusaurus.config.js) so future
versions are added by updating the releases array only.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f6dfdcb7-8228-4f61-9de7-3d9f51fbb694

📥 Commits

Reviewing files that changed from the base of the PR and between 55b0249 and 26246de.

📒 Files selected for processing (1)
  • docusaurus.config.js

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