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

Don't mention 'relaunch required' when updating plugins as part of an app upgrade #2224

Merged
merged 1 commit into from Jul 8, 2016

Conversation

pjrobertson
Copy link
Member

@pjrobertson pjrobertson commented Jul 3, 2016

When you do an app upgrade, Quicksilver first downloads the app, then checks for plugin updates and installs them. When installing these plugins a notification pops up saying the plugin is installed as per this line

The message 'Relaunch Required' really shouldn't display if Quicksilver is in the middle of an upgrade process (for which a relaunch would be required anyway)

If the core app is being updated, then the 'relaunch required' message which displays when plugins are installed should be omitted
@skurfer skurfer merged commit fe42d2d into master Jul 8, 2016
2 of 3 checks passed
@skurfer skurfer deleted the relaunch-required branch Jul 8, 2016
skurfer added a commit that referenced this issue Jul 8, 2016
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.

None yet

2 participants