-
Notifications
You must be signed in to change notification settings - Fork 39
Add 7.1.2 release notes #270
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
Caution Review failedThe pull request is closed. WalkthroughThe release notes for Unraid OS versions 7.1.0 and 7.1.1 were updated to clarify the data-loss risk related to the "mover empty disk" feature, with references to a fix in version 7.1.2. A new release notes file for version 7.1.2 was added, documenting the resolution of this issue and other minor updates. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant UnraidOS 7.1.0/7.1.1
participant UnraidOS 7.1.2
User->>UnraidOS 7.1.0/7.1.1: Uses "mover empty disk" feature
UnraidOS 7.1.0/7.1.1-->>User: Potential data-loss due to mishandled split levels
User->>UnraidOS 7.1.2: Upgrades system
UnraidOS 7.1.2-->>User: Issue resolved, normal mover and "mover empty disk" work as expected
Possibly related PRs
Poem
Tip ⚡️ Faster reviews with caching
Enjoy the performance boost—your workflow just got faster. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. 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: 1
♻️ Duplicate comments (1)
docs/unraid-os/release-notes/7.1.1.md (1)
12-12
: This issue also applies here: consider explicitly advising users not to use “Mover Empty Disk” until upgrading to 7.1.2 for clarity.
🧹 Nitpick comments (2)
docs/unraid-os/release-notes/7.1.0.md (1)
9-9
: Recommend adding user guidance to avoid using the feature
The note clarifies the risk but doesn’t explicitly instruct users to avoid using “Mover Empty Disk” until they can upgrade. Consider appending a caution like “Do not use the Mover Empty Disk feature until you have upgraded to 7.1.2.” to enhance clarity.docs/unraid-os/release-notes/7.1.2.md (1)
1-6
: Consider adding upgrade overview instructions
The “Upgrading” section is empty. For consistency with other release notes, consider adding a brief line such as “Upgrade via the webGUI update function or CLI (unraid-update
) to apply this release.” to guide users.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
docs/unraid-os/release-notes/7.1.0.md
(1 hunks)docs/unraid-os/release-notes/7.1.1.md
(1 hunks)docs/unraid-os/release-notes/7.1.2.md
(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: build
- GitHub Check: AWS Amplify Console Web Preview
### Known issues | ||
|
||
Please see the [7.1.0 release notes](7.1.0.md#known-issues). | ||
|
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.
Clarify known issues for this release
Currently, the “Known issues” section redirects to 7.1.0 rather than stating that all previous critical issues have been resolved in 7.1.2. Since the data-loss bug is fixed, consider explicitly marking “No known issues in this release.” or summarizing any remaining issues here to prevent confusion.
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