Skip to content

New Crowdin updates#469

Open
elibosley wants to merge 462 commits into
mainfrom
l10n_main
Open

New Crowdin updates#469
elibosley wants to merge 462 commits into
mainfrom
l10n_main

Conversation

@elibosley
Copy link
Copy Markdown
Member

@elibosley elibosley commented Apr 24, 2026

Summary by CodeRabbit

  • Documentation Updates

    • Major refresh of API docs (GraphQL Studio tips, streamlined auth and examples), expanded OIDC/SSO troubleshooting, new TPM licensing FAQ, clarified Unraid Connect/Remote Access and flash-backup/boot-device guidance, localized content and asset updates, and added 7.2.6 release notes.
  • UI/UX Improvements

    • Many pages now use collapsible sections for cleaner progressive disclosure; fixed heading/anchor rendering and improved clarity/readability across guides.

@coderabbitai coderabbitai Bot requested a review from ljm42 May 13, 2026 21:32
Copy link
Copy Markdown
Contributor

@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.

♻️ Duplicate comments (1)
i18n/de/docusaurus-plugin-content-docs/current/unraid-os/release-notes/7.3.0.md (1)

9-9: ⚠️ Potential issue | 🔴 Critical | ⚡ Quick win

Use MDX-safe comment anchors instead of classic heading IDs.

Both translated headings still use classic {#...} IDs, which violates repo rules and can break Crowdin MDX import parsing.

Proposed fix
-### Bekannte Probleme {`#known-issues`}
+### Bekannte Probleme {/* `#known-issues` */}

-### Rückschritte {`#rolling-back`}
+### Rückschritte {/* `#rolling-back` */}

As per coding guidelines: "For explicit heading anchors in MDX, especially translated docs, prefer Docusaurus's MDX-safe comment syntax: ### Heading {/* #stable-anchor */}. Do not use classic heading IDs like ### Heading {#stable-anchor} in translated MDX files; Crowdin's MDX import parser can reject them."

Also applies to: 30-30

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@i18n/de/docusaurus-plugin-content-docs/current/unraid-os/release-notes/7.3.0.md`
at line 9, Replace the classic MDX heading ID usage in the translated heading
"### Bekannte Probleme {`#known-issues`}" with Docusaurus MDX-safe comment anchors
by converting the trailing `{`#known-issues`}` into a comment-style anchor like
`{/* `#known-issues` */}`; locate the same pattern in any other translated
headings referenced (the second occurrence noted) and apply the same replacement
to ensure Crowdin MDX import compatibility and adherence to the repo guideline.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Duplicate comments:
In
`@i18n/de/docusaurus-plugin-content-docs/current/unraid-os/release-notes/7.3.0.md`:
- Line 9: Replace the classic MDX heading ID usage in the translated heading
"### Bekannte Probleme {`#known-issues`}" with Docusaurus MDX-safe comment anchors
by converting the trailing `{`#known-issues`}` into a comment-style anchor like
`{/* `#known-issues` */}`; locate the same pattern in any other translated
headings referenced (the second occurrence noted) and apply the same replacement
to ensure Crowdin MDX import compatibility and adherence to the repo guideline.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a0600be6-5198-48eb-bfe4-84d682efd288

📥 Commits

Reviewing files that changed from the base of the PR and between 9c40230 and 4924885.

📒 Files selected for processing (4)
  • i18n/de/docusaurus-plugin-content-docs/current/unraid-os/release-notes/7.3.0.md
  • i18n/es/docusaurus-plugin-content-docs/current/unraid-os/release-notes/7.3.0.md
  • i18n/fr/docusaurus-plugin-content-docs/current/unraid-os/release-notes/7.3.0.md
  • i18n/ja/docusaurus-plugin-content-docs/current/unraid-os/release-notes/7.3.0.md

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