fix(power): backport Power Button page collision fix to 7.3 - #2729
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
🔧 PR Test Plugin AvailableA test plugin has been generated for this PR that includes the modified files. Version: 📥 Installation Instructions:Install via Unraid Web UI:
Alternative: Direct Download
|
Summary
Backport the Power Options page-name collision fix to the Unraid 7.3.3 line on top of the reviewed 7.3 Power Button implementation.
Why This Exists
The built-in Power Button page uses the same filename stem as the Dynamix System Buttons plugin. PageBuilder keys pages by basename, so the plugin can replace the built-in Power Button tab.
This is the 7.3.3 follow-up to DST-200. It depends on the configurable Power Button backport tracked by OS-644 and PR #2715.
Resolution
Rename the built-in PowerButton.page to PowerModeButton.page. The page content, visible title, and tab position remain unchanged.
Reviewer Considerations
Behavior Changes
With Dynamix System Buttons installed, Settings > Power Options keeps both the Power Mode and Power Button tabs.
Implementation Summary
Verification
Risk
Low. The page content is unchanged. Only the page basename changes.
Linear Cross-links