Closed vjanssens closed 8 years ago
I'd love to see an integration with the GitHub API as well, we can grab the latest release tag using:
https://api.github.com/repos/gillesdemey/cumulus/releases/latest
and compare that with the current application version (from the package.json
).
That way we can show a nice dialog message in case the user is up-to-date.
Thanks for pointing me in the right direction, my latest commit contains a call to the GitHub API and showing a dialog box, as seen below.
As you can see, I haven't been able to change the icon in the dialog because i'm not sure what image to use and the dialog does not display at all when I add icon: __dirname + '/app/IconTemplate.png'
to mediaPlayer.js r.137. Maybe you could give it a try?
Another thing I'm not really happy about is mediaPlayer.js r.150. I would like to use the openExternal()
method on the mediaPlayer class, but this
is not bound to the mediaPlayer class anymore inside the promise. It works right now, so this would only be an improvement.
As for the version comparison, I've kept it as simple as possible. It just compares the tag from GitHub and the tag inside package.json and if they are equal, the app is up-to-date.
Can you let me know what you think? Thanks :)
No problem, if there is anything else please let me know :+1:
Sorry for the delay, I finally had some time to clean up the last bits today. I've changed the strings as per your request and added the Cumulus icon. The 'update available' dialog has a 'Download latest version' button which leads to https://github.com/gillesdemey/Cumulus/releases/latest.
LGTM! 👍
Added a 'check for updates' button that links to the latest release page of Cumulus on GitHub. Not the full auto-update feature issue #39 asks for, but it's a start which I think is useful.