Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

Remove Wizard(Action).done #1879

Merged
merged 1 commit into from
Apr 27, 2023
Merged

Remove Wizard(Action).done #1879

merged 1 commit into from
Apr 27, 2023

Conversation

jpnurmi
Copy link
Contributor

@jpnurmi jpnurmi commented Apr 27, 2023

It was used to finish the telemetry report and write it on the disk but that approach didn't work because Subiquity copies it to /target when it finishes installing and it's too late to write anything from the GUI when it quits. Also, writing any telemetry data from early exits is irrelevant because it doesn't end up stored anywhere anyway.

It was used to finish the telemetry report and write it on the disk but
that approach didn't work because Subiquity copies it to /target when it
finishes installing and it's too late to write anything from the GUI
when it gets closed. Also, writing any telemetry data from early exits
is irrelevant because it doesn't end up stored anywhere anyway.
@jpnurmi
Copy link
Contributor Author

jpnurmi commented Apr 27, 2023

For reference, _UbuntuDesktopInstallerWizardObserver.onDone was removed in:

Copy link
Member

@d-loose d-loose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jpnurmi jpnurmi merged commit 5b471f0 into canonical:main Apr 27, 2023
@jpnurmi jpnurmi deleted the wizard-done branch April 27, 2023 09:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants