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

(GH-769) Prevent automatic outdated package check #771

Merged
merged 4 commits into from
May 18, 2020

Conversation

mkevenaar
Copy link
Contributor

@mkevenaar mkevenaar commented Apr 29, 2020

Fixes #769

By enabeling this feature ChocolateyGUI will prevent
checking for outdated packages on start. An extra
button is added to check or refresh outdated packages.
@mkevenaar mkevenaar marked this pull request as ready for review April 30, 2020 20:16
@mkevenaar
Copy link
Contributor Author

Ready for review @gep13 @punker76 @AdmiringWorm @mwallner

Copy link
Member

@punker76 punker76 left a comment

Choose a reason for hiding this comment

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

LGTM

@mkevenaar mkevenaar changed the title WIP (GH-769) Prevent automatic outdated package check (GH-769) Prevent automatic outdated package check Apr 30, 2020
Copy link
Member

@AdmiringWorm AdmiringWorm left a comment

Choose a reason for hiding this comment

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

LGTM

@mkevenaar mkevenaar marked this pull request as draft May 4, 2020 21:07
gep13 and others added 2 commits May 4, 2020 14:08
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.
@punker76
Copy link
Member

punker76 commented May 4, 2020

After my changes...

chocogui_loading

The Show only outdated packages will now only be enabled
when it makes sense.
@mkevenaar mkevenaar marked this pull request as ready for review May 9, 2020 11:22
@mkevenaar
Copy link
Contributor Author

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.

Copy link
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

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

LGTM!

@gep13 gep13 merged commit 313fd0d into chocolatey:develop May 18, 2020
@gep13
Copy link
Member

gep13 commented May 18, 2020

@mkevenaar your changes have been merged, thanks for your contribution 👍

@gep13
Copy link
Member

gep13 commented May 18, 2020

@punker76 thanks also for your help on this!

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.

Add feature to toggle on/off the checking for outdated packages automatically
4 participants