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

[Testpilotcloud] Update channel warning does not reset dropdown selection #10251

Closed
2 tasks done
fmoc opened this issue Nov 9, 2022 · 10 comments · Fixed by #10609
Closed
2 tasks done

[Testpilotcloud] Update channel warning does not reset dropdown selection #10251

fmoc opened this issue Nov 9, 2022 · 10 comments · Fixed by #10609
Assignees
Labels
p2-high Escalation, on top of current planning, release blocker type:bug

Comments

@fmoc
Copy link
Contributor

fmoc commented Nov 9, 2022

Pre-submission Checks

  • I checked for similar issues, but could not find any. I also checked the closed issues. I could not contribute additional information to any existing issue.
  • I will take the time to fill in all the required fields. I know that the bug report may be dismissed otherwise due to lack of information.

Describe the bug

When changing the update channel in the settings, a dialog will pop up informing about the consequences. If the dialog is aborted, the selection in the dropdown is not reset.

The selection can be reset manually without triggering a new dialog.

Expected behavior

When aborting a dialog like this, the original state should be restored.

Steps to reproduce the issue

Change the update channel from "beta" to "stable" for instance.

Screenshots

No response

Logs

No response

Client version number

3.0.0-beta1 build 9006 on Ubuntu 22.04

Desktop environment (Linux only)

No response

Client package version and origin (Linux only)

No response

Installation path (Windows only)

No response

Server information

No response

Additional context

No response

@fmoc fmoc added the type:bug label Nov 9, 2022
@github-actions
Copy link

This issue was marked stale because it has been open for 30 days with no activity. Remove the stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label Dec 10, 2022
@fmoc fmoc removed the Stale label Dec 16, 2022
@fmoc fmoc added this to the 4.0 milestone Dec 16, 2022
@fmoc fmoc added the p4-low Low priority label Dec 16, 2022
@michaelstingl michaelstingl changed the title Update channel warning does not reset dropdown selection [Testpilotcloud] Update channel warning does not reset dropdown selection Jan 19, 2023
@TheOneRing
Copy link
Contributor

Can't reproduce on Windows.

@fmoc
Copy link
Contributor Author

fmoc commented Feb 13, 2023

Reproduced on Ubuntu 22.04 with the latest AppImage:

testpilotcloud 3.2.0.10137-daily20230213 c03e01
Qt Bibliotheken 5.15.5, OpenSSL 1.1.1q 5 Jul 2022
verwenden die Erweiterung für virtuelle Dateien: suffix
ubuntu-5.15.0-56-generic

@fmoc
Copy link
Contributor Author

fmoc commented Mar 17, 2023

Turns out the issue is localization. The settings dialog sets the entry by string and also uses string comparison to configure the current channel. When, e.g., using a German locale, the strings change from "stable" to "Stabil" and "beta" to "Beta", breaking the code. Using numeric indices to change back to the original value fixes the problem.

@Salipa-Gurung
Copy link
Contributor

Salipa-Gurung commented Mar 22, 2023

Version : testpilotcloud 4.0.0.10477-daily20230322 bb5d97

Reproduced on Ubuntu 22.04.2 Windows 10 pro
Initial channel: beta Screenshot from 2023-03-22 11-25-11 Screenshot from 2023-03-22 11-16-03
Selecting stable from dropdown and aborting dialog Screenshot from 2023-03-22 11-25-23 Screenshot from 2023-03-22 11-16-54
dropdown selection changed to: stable Screenshot from 2023-03-22 11-25-35 Screenshot from 2023-03-22 11-17-09
initial state is not restored ❌ initial state is not restored ❌

When initial channel is stable same behaviour is shown while selecting beta from dropdown and aborting dialog

CC: @fmoc @TheOneRing @HanaGemela

@saw-jan saw-jan reopened this Mar 22, 2023
@fmoc
Copy link
Contributor Author

fmoc commented Mar 22, 2023

Please test with a non-English locale.

@Salipa-Gurung
Copy link
Contributor

After changing language to Deutsch, dropdown selection is not resetting after aborting dialog box ❌

Screenshot from 2023-03-22 15-05-34 Screenshot from 2023-03-22 15-05-45 Screenshot from 2023-03-22 15-06-02

@TheOneRing TheOneRing added p3-medium Normal priority p2-high Escalation, on top of current planning, release blocker and removed p4-low Low priority p3-medium Normal priority labels Mar 22, 2023
@fmoc
Copy link
Contributor Author

fmoc commented Mar 22, 2023

Confirmed; worse than before, en locale is broken as well.

@fmoc
Copy link
Contributor Author

fmoc commented Apr 5, 2023

Fix provided in #10627.

@saw-jan
Copy link
Member

saw-jan commented Apr 11, 2023

Tested with testpilotcloud 4.0.0.10610-daily20230411 a8eb35

Screenshot from 2023-04-11 16-08-28 Screenshot from 2023-04-11 16-08-22 Screenshot from 2023-04-11 16-08-15
Default state Change the channel and Cancel Reverts to previous state ✔️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p2-high Escalation, on top of current planning, release blocker type:bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants