fix interactive notifications icon + exclude platform specific plugins from build #591
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
pass appID to node notifier in release mode (assuming release has updated binaries from #600)
Fix #479 (see my comment there for more info about upstream pr's that are almost one year old)
old behavior:

new behavior:

The problem was that setting an
appID(which makes the icon and appName show up) would make the buttons not work (returning undefined instead of index of clicked button)This was caused because of a problem in a dependency of a dependency (node-notifier->snoretoast)
and the pr with the fix has still yet to be merged many months later, so we now use a custom compiled snoretoast that include said fix
package.json changes
taskbar-mediacontrolplugin added only to windows distributionstouchbarplugin added only to macOS distributions