Skip to content

Conversation

@Minh141120
Copy link
Member

@Minh141120 Minh141120 commented Sep 25, 2025

This pull request updates the redirect rules in the docs/_redirects file to improve URL handling and ensure more consistent user navigation. The main changes involve adding a new redirect and refining an existing one.

Redirect improvements:

  • Added a redirect from /about to /handbook with a 302 status to match the existing redirect from /about/, ensuring both URLs are handled consistently.
  • Updated the /mcp/ redirect to /mcp (removing the trailing slash) to standardize the URL format and prevent duplicate redirects.

Important

Update redirect rules in docs/_redirects to add /about redirect and refine /mcp/ redirect.

  • Redirects:
    • Added redirect from /about to /handbook with 302 status to match existing /about/ redirect.
    • Updated /mcp/ redirect to /mcp to standardize URL format and prevent duplicates.

This description was created by Ellipsis for 20c8991. You can customize this summary. It will automatically update as commits are pushed.

@Minh141120 Minh141120 self-assigned this Sep 25, 2025
@Minh141120 Minh141120 requested a review from hiento09 September 25, 2025 13:54
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 20c8991 in 56 seconds. Click for details.
  • Reviewed 18 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. docs/_redirects:1
  • Draft comment:
    Added '/about' redirect appears intentional to cover both '/about' and '/about/' cases. Confirm that having both rules is desired to avoid redundancy.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None
2. docs/_redirects:77
  • Draft comment:
    Updated '/mcp/' to '/mcp' standardizes the URL, but consider if a rule for '/mcp/' should be preserved for users who might include the trailing slash.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_ZjyS7LNQ0UarJ9T1

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@github-actions
Copy link
Contributor

Preview URL: https://f0f5d3e4.docs-9ba.pages.dev

@Minh141120 Minh141120 merged commit 75396db into dev Sep 25, 2025
2 checks passed
@Minh141120 Minh141120 deleted the docs/update-redirects branch September 25, 2025 13:57
@github-project-automation github-project-automation bot moved this to QA in Jan Sep 25, 2025
@github-actions github-actions bot added this to the v0.7.0 milestone Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants