-
Notifications
You must be signed in to change notification settings - Fork 39
Add 7.1.0-beta.3 release notes #255
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
WalkthroughThe release notes for Unraid OS have been updated from version 7.1.0-beta.2 to 7.1.0-beta.3. The documentation now includes a new section titled "Unraid Connect" that warns users against uninstalling its plugin during this release, as doing so will break the webGUI until a manual reboot is performed. Additional details about known issues—such as nchan out of memory errors—are provided, along with fixes addressing ZFS device replacement and VLAN configuration. The "Misc" section has been renamed "WebGUI" with updates on CSS responsiveness and further user guidance. No public API changes were made. Changes
Possibly related PRs
Poem
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
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 (2)
docs/unraid-os/release-notes/7.1.0.md (2)
139-139
: Punctuation Suggestion for Enhanced Readability in WebGUI Section
The sentence on line 139 reads:"As a step toward making the webGUI responsive, we have reworked the CSS. For the most part this should not be noticeable aside from some minor color adjustments. ..."
For clarity and proper introductory pacing, consider adding a comma after "For the most part".-As a step toward making the webGUI responsive, we have reworked the CSS. For the most part this should not be noticeable aside from some minor color adjustments. We expect that most plugins should be fine as well, although plugin authors may want to review [this documentation](https://github.com/unraid/webgui/blob/master/emhttp/plugins/dynamix/styles/themes/README.md). [-beta.3] +As a step toward making the webGUI responsive, we have reworked the CSS. For the most part, this should not be noticeable aside from some minor color adjustments. We expect that most plugins should be fine as well, although plugin authors may want to review [this documentation](https://github.com/unraid/webgui/blob/master/emhttp/plugins/dynamix/styles/themes/README.md). [-beta.3]🧰 Tools
🪛 LanguageTool
[typographical] ~139-~139: Consider adding a comma after this introductory phrase.
Context: ... we have reworked the CSS. For the most part this should not be noticeable aside fro...(FOR_THE_MOST_PART_COMMA)
141-141
: Punctuation Correction in WebGUI Section
The sentence on line 141 currently reads:"If you notice alignment issues or obvious color problems in any official theme please let us know."
Adding a comma after "official theme" will improve the flow:-If you notice alignment issues or obvious color problems in any official theme please let us know. +If you notice alignment issues or obvious color problems in any official theme, please let us know.🧰 Tools
🪛 LanguageTool
[typographical] ~141-~141: It seems that a comma is missing.
Context: ...ous color problems in any official theme please let us know. ### Live Updates Paused ...(IF_PLEASE_COMMA)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for pro 8000 cessing (1)
docs/unraid-os/release-notes/7.1.0.md
(4 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/unraid-os/release-notes/7.1.0.md
[typographical] ~139-~139: Consider adding a comma after this introductory phrase.
Context: ... we have reworked the CSS. For the most part this should not be noticeable aside fro...
(FOR_THE_MOST_PART_COMMA)
[typographical] ~141-~141: It seems that a comma is missing.
Context: ...ous color problems in any official theme please let us know. ### Live Updates Paused ...
(IF_PLEASE_COMMA)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: build
- GitHub Check: AWS Amplify Console Web Preview
🔇 Additional comments (6)
docs/unraid-os/release-notes/7.1.0.md (6)
1-1
: Version Header Updated Correctly
The version header is now updated to “7.1.0-beta.3” with an appropriate release date. Please double-check that the date and version meet your release standards.
5-5
: Updated Release Description
The new release description clearly highlights the new features such as wireless networking, TrueNAS pool import, VM enhancements, and early steps toward a responsive webGUI. The description reads well and provides comprehensive information.
11-14
: Addition of Unraid Connect Section
The new "Unraid Connect" section warns users not to uninstall the plugin, which is critical since doing so can break the webGUI until a manual reboot. This guidance is clear and actionable.
21-24
: nchan Out of Memory Issues Section
The new section on “nchan out of memory issues” explains the issue and instructs users to seek help via a forum thread if they continue to face problems. Ensure that support channels or further documentation links are readily available for users.
70-70
: VLAN Apply Button Fix Noted
The fix enabling the Apply button when adding a VLAN is clearly documented with the appropriate beta marker. This change should help prevent configuration issues in the network settings.
149-152
: New “Other WebGUI changes” Section Added
The addition of the “Other WebGUI changes” section with the fix to automatically restart nginx if nchan runs out of shared memory is clearly stated. This should help mitigate potential service interruptions.
This pull request is automatically being deployed by Amplify Hosting (learn more). |
Before Submitting This PR, Please Ensure You Have Completed The Following:
Summary by CodeRabbit
Documentation
Bug Fixes
New Features