Skip to content

Add go link for convert-reiser-and-xfs - #314

Merged
ljm42 merged 1 commit into
mainfrom
add-go-link
Sep 10, 2025
Merged

Add go link for convert-reiser-and-xfs#314
ljm42 merged 1 commit into
mainfrom
add-go-link

Conversation

@ljm42

@ljm42 ljm42 commented Sep 10, 2025

Copy link
Copy Markdown
Member

Before Submitting This PR, Please Ensure You Have Completed The Following:

  1. Are internal links to wiki documents using relative file links?
  2. Are all new documentation files lowercase, with dash separated names (ex. unraid-os.mdx)?
  3. Are all assets (images, etc), located in an assets/ subfolder next to the .md/mdx files?
  4. Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  5. Is the build succeeding?

Summary by CodeRabbit

  • Documentation
    • Added a redirect from the legacy URL “/go/convert-reiser-and-xfs/” to the updated documentation on converting to a new file system type. This preserves existing bookmarks, prevents 404s, and improves navigation continuity.
    • Enhances discoverability by guiding users to the current, consolidated content path for file system conversion details within the Unraid OS docs.

@coderabbitai

coderabbitai Bot commented Sep 10, 2025

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

A new redirect entry was added to the Docusaurus client redirects configuration, mapping "/go/convert-reiser-and-xfs/" to "/unraid-os/using-unraid-to/manage-storage/file-systems/#converting-to-a-new-file-system-type". No other configuration or exported function signatures were modified.

Changes

Cohort / File(s) Summary of edits
Docs redirects
docusaurus.config.js
Added a redirect mapping from /go/convert-reiser-and-xfs/ to /unraid-os/using-unraid-to/manage-storage/file-systems/#converting-to-a-new-file-system-type within the client redirects plugin configuration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks (2 passed, 1 warning)

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The current description only includes the unmodified submission checklist template and lacks any summary of the actual changes, context for why the redirect was added, or confirmation that the checklist items have been completed. Please update the description to include a dedicated summary of the change (including the new redirect mapping), justify its purpose, and mark off or address each item in the submission checklist to confirm links, naming conventions, assets, duplicate PRs, and build status.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title “Add go link for convert-reiser-and-xfs” accurately and concisely summarizes the primary change by specifying that a new go link is being added for the convert-reiser-and-xfs page, making it clear and specific for reviewers scanning the history.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

Poem

I hop through docs with gentle cheer,
A path now points from there to here.
Old trails nudge to fresher nests,
Redirects guide the reader’s quests.
Thump-thump—links aligned and neat,
Carrots cached, a tidy feat! 🥕

✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch add-go-link

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.

@aws-amplify-us-west-2

Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-314.d1o5s833kbd339.amplifyapp.com

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 0

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

334-334: Anchor verified; optional alias go-links recommended
The slug #converting-to-a-new-file-system-type exists in docs/unraid-os/using-unraid-to/manage-storage/file-systems.md (line 132), and no duplicate go-links were found. Optionally add:

 { from: "/go/convert-reiser-and-xfs/", to: "/unraid-os/using-unraid-to/manage-storage/file-systems/#converting-to-a-new-file-system-type" },
+{ from: "/go/convert-reiser-to-xfs/",    to: "/unraid-os/using-unraid-to/manage-storage/file-systems/#converting-to-a-new-file-system-type" },
+{ from: "/go/convert-reiserfs-to-xfs/", to: "/unraid-os/using-unraid-to/manage-storage/file-systems/#converting-to-a-new-file-system-type" },
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cf130c2 and 73a6482.

📒 Files selected for processing (1)
  • docusaurus.config.js (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: AWS Amplify Console Web Preview

@ljm42
ljm42 merged commit 284e940 into main Sep 10, 2025
3 checks passed
@ljm42
ljm42 deleted the add-go-link branch September 10, 2025 17: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.

1 participant