feat: Add Account app MFA instructions#492
Conversation
📝 WalkthroughWalkthroughThis PR introduces a new documentation page for multi-factor authentication (MFA) on Unraid accounts. The page covers prerequisites, step-by-step enable/disable flows, and common troubleshooting scenarios such as setup failures, rejected codes, and lost authenticator recovery. ChangesMulti-Factor Authentication Documentation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
unraid-docs | 6c63daa | Commit Preview URL Branch Preview URL |
May 27 2026, 06:31 PM |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/unraid-account/multi-factor-authentication.mdx (1)
40-42: 💤 Low valueConsider varying the sentence structure for better readability.
All three troubleshooting bullets begin with "If," creating a repetitive pattern. Rephrasing one improves flow.
📝 Suggested rewording
- If setup fails, start again from the **MFA** tab in **Account Settings** and scan the new QR code. -- If a code is rejected, wait for the next code in your authenticator app and try again. +- When a code is rejected, wait for the next code in your authenticator app and try again. - If you no longer have access to your authenticator app, contact support for help recovering access to your account.🤖 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 `@docs/unraid-account/multi-factor-authentication.mdx` around lines 40 - 42, The three troubleshooting bullets in multi-factor-authentication.mdx are repetitive because they all start with "If"; rephrase one or two bullets to vary sentence structure for better flow — for example, change the first to "Should setup fail, return to the MFA tab in Account Settings and scan the new QR code," or make the third active like "Contact support to recover access if you no longer have your authenticator app" — update the three list items accordingly to avoid starting each sentence with "If" while preserving the original meaning.
🤖 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.
Nitpick comments:
In `@docs/unraid-account/multi-factor-authentication.mdx`:
- Around line 40-42: The three troubleshooting bullets in
multi-factor-authentication.mdx are repetitive because they all start with "If";
rephrase one or two bullets to vary sentence structure for better flow — for
example, change the first to "Should setup fail, return to the MFA tab in
Account Settings and scan the new QR code," or make the third active like
"Contact support to recover access if you no longer have your authenticator app"
— update the three list items accordingly to avoid starting each sentence with
"If" while preserving the original meaning.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: a0d3ac98-6149-4503-bc87-7d1a988fc2df
📒 Files selected for processing (1)
docs/unraid-account/multi-factor-authentication.mdx
Before Submitting This PR, Please Ensure You Have Completed The Following:
Summary by CodeRabbit