-
Notifications
You must be signed in to change notification settings - Fork 252
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
(GH-769) Prevent automatic outdated package check #771
Conversation
By enabeling this feature ChocolateyGUI will prevent checking for outdated packages on start. An extra button is added to check or refresh outdated packages.
46de272
to
d58a9ff
Compare
Ready for review @gep13 @punker76 @AdmiringWorm @mwallner |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Worked on this during the stream, trying to decide the best way to implement things. Went backwards and forwards on a few topics, but think we are getting closer to final result.
…e invalidating the command stuff Force invalidating the command stuff. This helps us to prevent disabled buttons after executing this routine. But IMO it has something to do with Caliburn.
The Show only outdated packages will now only be enabled when it makes sense.
I have made a small change to the translation to match the feature better. I also prevented the "Show Only Packages with Updates" to be enabled when we haven't checked for outdated packages. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@mkevenaar your changes have been merged, thanks for your contribution 👍 |
@punker76 thanks also for your help on this! |
Fixes #769