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

Fix update channel reset #10627

Merged
merged 1 commit into from
Apr 6, 2023
Merged

Fix update channel reset #10627

merged 1 commit into from
Apr 6, 2023

Conversation

fmoc
Copy link
Contributor

@fmoc fmoc commented Mar 23, 2023

We need to keep our own copy of the old index to be able to reliably reset the selection if the dialog is rejected.

@fmoc fmoc added this to the Desktop Client 4.0 milestone Mar 23, 2023
@fmoc fmoc requested a review from TheOneRing March 23, 2023 01:55
@TheOneRing
Copy link
Contributor

We could be more elegant and use the data role for the combobox.
(store stable/beta as data role and look up their index, also use their value to set the channel)
That way we would not need to store indices.

We need to keep our own copy of the old index to be able to reliably reset the selection if the dialog is rejected.
@fmoc fmoc force-pushed the work/fix-language-picker branch from b055bb0 to 773c885 Compare April 5, 2023 19:07
@fmoc
Copy link
Contributor Author

fmoc commented Apr 5, 2023

Fixed.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 5, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@fmoc fmoc merged commit 089c91e into master Apr 6, 2023
@delete-merged-branch delete-merged-branch bot deleted the work/fix-language-picker branch April 6, 2023 10:21
@saw-jan saw-jan requested review from saw-jan and removed request for saw-jan April 11, 2023 09:23
@saw-jan
Copy link
Member

saw-jan commented Apr 11, 2023

Tested in #10251 (comment)

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.

4 participants