Skip to content
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

Integrate with distinst's new autoinstall features #126

Merged
merged 9 commits into from May 29, 2018

Conversation

mmstick
Copy link
Member

@mmstick mmstick commented May 15, 2018

  • Opens the door to retaining home and supporting refresh installation options. The logic is implemented here in the frontend, but will also need a UI to take advantage of it (listing refresh options & setting retain_home in the config).
  • The backend now handles more of what the DiskView was previously doing, so when a disk is selected, the EraseOption for that button will be stored and applied in the ProgressView.
  • For each entry into the DiskView, a global Distinst.Disks will be used for generating the list in the view, and Distinst.device_layout_hash() will be used to know when the Disks object needs to be regenerated.
  • Removes the default_disk_configuration() method due to the backend now supporting it via applying the EraseOption to the disks object.
  • Removes the recovery_disk_configuration() method due to the backend now supporting it via applying the RecoveryOption to the disk object.
  • Merges elementary@f2a6aec

Requires pop-os/distinst#95

@mmstick mmstick force-pushed the autoinstall branch 2 times, most recently from fc22edf to dedfaca Compare May 18, 2018 21:50
@mmstick
Copy link
Member Author

mmstick commented May 23, 2018

@brs17 @jackpot51 This is also ready

@jackpot51 jackpot51 merged commit 63d3a7e into master_bionic May 29, 2018
@jackpot51 jackpot51 deleted the autoinstall branch May 29, 2018 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants