Handle need restart after installation for winget

[panther7]

#625

Added some new texts, but question is, are we really need dialog?

@marticliment

[marticliment]

Instead of showing a dialog, what do you think about showing only a push notification, with the globals.trayIcon.showMessage function?

[marticliment]

I think this dialogs might be a little bit intrusive, especially when bulk-updating packages

[panther7]

I think this dialogs might be a little bit intrusive, especially when bulk-updating packages

I agree, but now, show "error" dialog.

Should I just rewrite it in a text message like "installed successfully"?

[marticliment]

The current behaviour is:

1. If failed, show a message.
2. If succeeded, throw a notification.

What I would do is change the text of that "{0} installation succeeded" for "A restart is required to finish {0} installation", as well as addding a visual indicator in the ui telling the user to restart its PC.

If you don't understand how the error message class works, (it is a little bit messy), just let me know and I'll implement this feature.

Don't worry for the visual indicator, since I am working on a base class for them, since I need to implement one for one of the chocolatey features.

[panther7]

Done, new strings:

• {action} was successfully!
• Restart your PC to finish installation

Code is now more "simplify".

[marticliment]

I have added new restart icons, so it looks visually different