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

Github as NSIS update server #868

Closed
zh99998 opened this issue Nov 2, 2016 · 6 comments · May be fixed by qcif/data-curator#563
Closed

Github as NSIS update server #868

zh99998 opened this issue Nov 2, 2016 · 6 comments · May be fixed by qcif/data-curator#563

Comments

@zh99998
Copy link

zh99998 commented Nov 2, 2016

currently NSIS updater can't use github as update server.

as #529 (comment)

@develar develar changed the title feature request: use Github as NSIS update server. Github as NSIS update server Nov 3, 2016
@develar develar closed this as completed in 9e18cb1 Nov 4, 2016
@develar
Copy link
Member

develar commented Nov 4, 2016

@oopets
Copy link

oopets commented Nov 6, 2016

@develar Hi, I use github as a provider and latest.yml is not generated, only .exe file is generated and uploaded. Thus, there is an error "Error: Cannot find latest.yml in the latest release artifacts: Error: 404 Not Found" while app checks for updates.

@develar develar reopened this Nov 6, 2016
@develar
Copy link
Member

develar commented Nov 6, 2016

@integrable Thanks, fixed,

@develar develar closed this as completed in 9d31b42 Nov 6, 2016
@oopets
Copy link

oopets commented Nov 6, 2016

@develar Thank you for the quick fix! Works for me on Windows 10.
So far I have also found few issues:

  1. ReleaseNotes variable in update-downloaded handler is Null
  2. latest.yml is not automatically uploaded to GitHub (I do it manually)
  3. The name of a file uploaded to GitHub is based on "name" setting from app/package.json, while the .exe name is actually "productName"+ "-Setup-", also "path" in latest.yml would be -Setup-.exe, so it has to be corrected manually, otherwise the file will not be found during update download .

@develar develar reopened this Nov 6, 2016
develar added a commit to develar/electron-builder that referenced this issue Nov 7, 2016
develar added a commit to develar/electron-builder that referenced this issue Nov 7, 2016
@develar develar closed this as completed in e920584 Nov 7, 2016
@develar
Copy link
Member

develar commented Nov 7, 2016

ReleaseNotes variable in update-downloaded handler is Null

Please file issue. Why do you need release notes?

@oopets
Copy link

oopets commented Nov 7, 2016

@develar In fact, the issue is not critical for me. In my case, the app just shows release notes when the new version of app is downloaded:

 autoUpdater.addListener("update-downloaded", function (event, releaseNotes, releaseName, releaseDate, updateURL) {
      const windows = BrowserWindow.getAllWindows();
      const cancelIndex = 1; 
      const index = dialog.showMessageBox(windows[0], {
        type: 'info',
        buttons: ['Restart', 'Later'],
        title: "Update available",
        message: 'The new version has been downloaded.',
        detail: releaseName + (releaseNotes ? "\n\n" + releaseNotes : '')
      });

      if (index === cancelIndex) {
        return;
      }
      autoUpdater.quitAndInstall();
    });

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