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

Check for Updates On Startup does not toggle off #978

Closed
redromnon opened this issue Feb 17, 2022 · 3 comments · Fixed by #980
Closed

Check for Updates On Startup does not toggle off #978

redromnon opened this issue Feb 17, 2022 · 3 comments · Fixed by #980
Assignees
Labels
bug:confirmed Something isn't working.

Comments

@redromnon
Copy link
Collaborator

IMPORTANT! Do not erase this template!

Is this a game issue? If YES please use our Discord to get help.

Open here only issues with Heroic like: Heroic doesn't open, some button is out of place, Heroic closes unexpectedly, installs got interrupted, etc.

Try to get help on our Wiki first: https://github.com/Heroic-Games-Launcher/HeroicGamesLauncher/wiki/Troubleshooting

Describe the bug
The checkbox Check for Updates on Startup, under General (Global) Settings, always toggles itself on despite turning it off. When I reopen the app or move to another tab/window and come back to the General Settings window, the checkbox is turned on automatically. I believe the toggle's state doesn't get saved and thus leads to this issue.

Add Logs
None

To Reproduce
Steps to reproduce the behavior:

  1. Go to Settings
  2. Click General
  3. Under Global Settings
  4. Uncheck the Check for Updates on Startup checkbox

Expected behavior
The checkbox should be always unchecked when done so.

Screenshots
Check_for_Updates_Heroic

Desktop (please complete the following information):

  • OS: Pop_OS 20.04
  • Heroic Version - 2.1.1 Rayleigh

Additional context
None

@redromnon redromnon changed the title Check for Updates On Startup does toggle turn off Check for Updates On Startup does not toggle off Feb 17, 2022
@Nocccer Nocccer self-assigned this Feb 17, 2022
@Nocccer Nocccer added the bug:confirmed Something isn't working. label Feb 17, 2022
@Nocccer
Copy link
Collaborator

Nocccer commented Feb 17, 2022

Can reproduce it. Will try to fix it.

@Nocccer Nocccer linked a pull request Feb 17, 2022 that will close this issue
4 tasks
@flavioislima
Copy link
Member

Can reproduce it. Will try to fix it.

I believe the issue is on this line:

Because this settings' default is true, but on this line, it starts as false. So there is some conflict.
changing here to true should fix the issue.

@Nocccer
Copy link
Collaborator

Nocccer commented Feb 17, 2022

@flavioislima already fixed it for ~2h

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug:confirmed Something isn't working.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants