-
Notifications
You must be signed in to change notification settings - Fork 12
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
Limit reset partitioning #237
Conversation
96a966b
to
3b45086
Compare
7985961
to
51bdf80
Compare
fc34a72
to
b5b37ed
Compare
If the type matches, (MANUAL vs AUTOMATIC) and the devices also match then we can reuse the partitioning. Reset the partitioning always when entering the Installation Destination page.
Put this in a custom hook to increase component readability.
Identifying locked devices can be done prior to partitioning creation. After unlocking the devices we had to re-create the partitioning anyway. For implementing this change, adjust getLockedLUKSDevices helper to parse selectedDisks instead of the partitioning requests for identifying locked LUKS devices.
b5b37ed
to
f518755
Compare
Fixed issues and rebased. |
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.
Looks good to me, thank you.
const initialRequests = getInitialRequests(requests, mountPointConstraints); | ||
setUnappliedRequests(initialRequests); | ||
|
||
setIsFormValid(getRequestsValid(initialRequests, deviceData)); | ||
}, [deviceData, setIsFormValid, partitioningDataPath, requests, mountPointConstraints]); | ||
}, [deviceData, setIsFormValid, partitioning.path, requests, isLoadingPartitioning, unappliedRequests, mountPointConstraints]); |
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.
The partitioning.path
dependency is listed here so that the hook is run in case applied partitioning changes, right?
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.
Correct, so if we have a new partitioning we need to recalculate the displayed requests.
No description provided.