Skip to content

EPMRPP-113922 || Added trailing slashes#1098

Open
maria-hambardzumian wants to merge 1 commit intodevelopfrom
feature/EPMRPP-113922-Added-trailing-slashes
Open

EPMRPP-113922 || Added trailing slashes#1098
maria-hambardzumian wants to merge 1 commit intodevelopfrom
feature/EPMRPP-113922-Added-trailing-slashes

Conversation

@maria-hambardzumian
Copy link
Copy Markdown
Contributor

@maria-hambardzumian maria-hambardzumian commented Mar 27, 2026

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated website URL structure to consistently use trailing slashes across all navigation links and pages for improved consistency.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 27, 2026

Walkthrough

Enabled trailing slashes in Docusaurus configuration and updated footer navigation links to include trailing slashes for URL consistency across the site.

Changes

Cohort / File(s) Summary
Configuration Update
docusaurus.config.js
Enabled trailingSlash: true in Docusaurus config and updated three footer navigation links (installation-steps, configuration, developers-guides) to include trailing slashes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • AmsterGet
  • pressayuliya

Poem

🐰 With trailing slashes now in place, hooray!
Our URLs dance with style and grace.
Configuration set, links all neat,
Consistency makes the site complete.
Forward slashes hop in formation fine,
Every path now ends with a line!

🚥 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 'Added trailing slashes' directly and clearly describes the main change made in the pull request—enabling trailing slashes in Docusaurus config and updating footer links.
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
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/EPMRPP-113922-Added-trailing-slashes

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.

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)

23-23: Normalize internal redirect targets to canonical style with trailing slash.

With trailingSlash: true enabled at line 23, the 60+ internal redirect to targets throughout the redirect rules (lines 236–612) should consistently include trailing slashes. For example, /installation-steps should be /installation-steps/, /dashboards-and-widgets/CumulativeTrendChart should be /dashboards-and-widgets/CumulativeTrendChart/, etc. This ensures redirect destinations match the canonical URL format and avoids unnecessary URL normalization hops.

Examples of targets lacking trailing slash

Line 236: /installation-steps
Line 240: /features
Line 244: /releases
Line 248: /user-account
Line 252: /work-with-reports
...and 55+ additional targets throughout the redirect rules.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@docusaurus.config.js` at line 23, The site config enables trailingSlash
(trailingSlash: true) but many redirect rule targets are missing the trailing
slash; update every redirect object in the redirects rules so each "to" value
uses the canonical trailing-slash form (e.g., change "/installation-steps" to
"/installation-steps/") across the redirects array so destinations match the
canonical URLs and avoid normalization hops.
🤖 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`:
- Line 23: The site config enables trailingSlash (trailingSlash: true) but many
redirect rule targets are missing the trailing slash; update every redirect
object in the redirects rules so each "to" value uses the canonical
trailing-slash form (e.g., change "/installation-steps" to
"/installation-steps/") across the redirects array so destinations match the
canonical URLs and avoid normalization hops.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 29beb3ee-313f-4400-8011-1b72407d511b

📥 Commits

Reviewing files that changed from the base of the PR and between 2fc2c56 and 5499654.

📒 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