Skip to content

Fixed spelling mistake for the scrub/balance message when the pool is unmounted#2558

Merged
limetech merged 1 commit intounraid:masterfrom
Bushingz:fix/spelling-mistake-scrub
Feb 27, 2026
Merged

Fixed spelling mistake for the scrub/balance message when the pool is unmounted#2558
limetech merged 1 commit intounraid:masterfrom
Bushingz:fix/spelling-mistake-scrub

Conversation

@Bushingz
Copy link
Contributor

@Bushingz Bushingz commented Feb 25, 2026

image

Summary by CodeRabbit

Bug Fixes

  • Corrected spelling of "filesystem" in three user-visible text strings across the Device Info interface, including tooltip text for Balance and Scrub actions, and status messaging for ZFS pool scrub controls.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between c7c9e76 and 9240ef7.

📒 Files selected for processing (1)
  • emhttp/plugins/dynamix/DeviceInfo.page

Walkthrough

A single-file correction that fixes three instances of a misspelled term "filesyestem" to "filesystem" in user-visible tooltip and message text within the DeviceInfo page. No functional or logic changes.

Changes

Cohort / File(s) Summary
Spelling Corrections
emhttp/plugins/dynamix/DeviceInfo.page
Corrects misspelled term "filesyestem" to "filesystem" in three tooltip texts for balance action, scrub action, and ZFS pool scrub controls.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A typo hopped across the screen,
Where "filesyestem" had been seen,
With whiskers twitched, we set things right,
Now "filesystem" shines so bright! ✨🐰

🚥 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 accurately describes the main change: fixing spelling mistakes in scrub/balance messages related to an unmounted pool.
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 unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@elibosley elibosley added the 7.3 label Feb 26, 2026
Copy link
Member

@elibosley elibosley left a comment

Choose a reason for hiding this comment

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

This looks great. Thank you

@limetech limetech merged commit c435862 into unraid:master Feb 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants