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

Allow to start app after silent install #1545

Closed
maczikasz opened this issue May 15, 2017 · 6 comments · May be fixed by qcif/data-curator#563
Closed

Allow to start app after silent install #1545

maczikasz opened this issue May 15, 2017 · 6 comments · May be fixed by qcif/data-curator#563

Comments

@maczikasz
Copy link
Contributor

  • Version: latest
latest
  • Target: nsis

Hi, we would like to do silent updates on our app, but using our one click installer that's at the moment not possible. We are calling quitAndInstall on the update-downloaded trigger, which does not start in silent mode.

The other thing is that we would like to do it seemlessly so the app should start after a silent install.

There are two things there then, to allow to specify on quit and install to do a silent one , or an other method, and the other thing is to be able to specify for the installer a flag at runtime or compile time that overrides the if in :

@develar
Copy link
Member

develar commented May 16, 2017

If you call quitAndInstall explicitly (not auto install after user update), why installation should be silent?

@maczikasz
Copy link
Contributor Author

what we want to achieve is to install as soon as the update is downloaded then start the app again, and this to be completely silent

Our app normally runs in the background (on the tray, this way we can achieve silent updates without the user being notified) if there is a better way to achieve this then pls tell me

@maczikasz
Copy link
Contributor Author

My understanding how this works at the moment:

Install on app quit -> silent mode -> app wont start automatically
Install by shutting down the app -> non silent (now silent possible) -> app starts automatical

@develar
Copy link
Member

develar commented May 22, 2017

App will be not started automatically on silent install. As you pointed (

). So, I think, this issue should be opened until it is not fixed.

@maczikasz maczikasz reopened this May 22, 2017
@maczikasz
Copy link
Contributor Author

I closed by accident, probably clicked on the wrong button, sry

@tiangolo
Copy link

I just had an issue using TypeScript, it seems like I have to call autoUpdater.quitAndInstall(isSilent: boolean) with an explicit false for the isSilent parameter, as in autoUpdater.quitAndInstall(false).

Otherwise I see a Supplied parameters do not match any signature of call target..


But as I understand, that flag should have a default to false and be needed only for Windows / NSIS, right? That's what I infer from this comment: https://github.com/electron-userland/electron-builder/blob/master/packages/electron-updater/src/AppUpdater.ts#L265

It seems to have the default false in the specific code for NSIS in https://github.com/electron-userland/electron-builder/blob/master/packages/electron-updater/src/NsisUpdater.ts#L85

But not in the general AutoUpdater: https://github.com/electron-userland/electron-builder/blob/master/packages/electron-updater/src/AppUpdater.ts#L267

Am I misunderstanding something or maybe does the AutoUpdater need to have that default parameter too?

maczikasz pushed a commit to maczikasz/electron-builder that referenced this issue Jun 20, 2017
If --force-run flag is specified on installer then will start the app on silent install
Exposed forceRun flag on NSIS updater quit and install, default value is false

electron-userland#1545
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants