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

Auto update application #35

Merged
merged 10 commits into from
Apr 28, 2020
Merged

Auto update application #35

merged 10 commits into from
Apr 28, 2020

Conversation

pverscha
Copy link
Member

@pverscha pverscha commented Mar 5, 2020

Automatically update the desktop application when a new version is available. The new version is downloaded and installed automatically in the background without the need for user intervenience.

The user is informed about the download process with a snackbar at the bottom of the application:
image

In case of a successful installation the user is greeted with a different message:
image

Errors that may occur during application updates are intercepted and presented to the user:
image

@pverscha pverscha added the feature Describes a completely new feature label Mar 5, 2020
@pverscha pverscha self-assigned this Mar 5, 2020
@pverscha
Copy link
Member Author

pverscha commented Mar 6, 2020

This PR also includes a GitHub action that automatically builds a new version of our application after each push to master.

@pverscha
Copy link
Member Author

pverscha commented Mar 6, 2020

This PR should not be merged before auto-updating works on macOS. For this to work, we are required to notarise our application. This can only be done by Apple Developers (which requires us to pay a 100$ registration fee each year). Notarising our app also removes the GateKeeper warning that appears when users try to install Unipept Desktop.

For Windows, we are also required to sign the app using an EV-certificate in order to get rid of the "Windows protected your PC" message that obstructs installation.

Both notarisation and signing are not mandatory, but will discourage users to install the app.

@pverscha pverscha marked this pull request as ready for review April 28, 2020 06:46
@pverscha pverscha changed the base branch from master to develop April 28, 2020 06:47
@pverscha pverscha merged commit 01dfc33 into develop Apr 28, 2020
@pverscha pverscha mentioned this pull request Apr 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Describes a completely new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant