-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
CheckForUpdateImpl: Couldn't write out staging user ID #1171
Comments
Is it possible to use NSIS? https://github.com/electron-userland/electron-builder/wiki/Auto-Update |
@develar I believe there is no workaround for above error 😞
|
Feel free to ask me if you will have any troubles with NSIS. |
@develar so here is an update on my autoupdater function - 'use strict';
const {app, dialog} = require('electron');
const {autoUpdater} = require('electron-updater').autoUpdater;
function appUpdater() {
// Log whats happening
const log = require('electron-log');
log.transports.file.level = 'info';
autoUpdater.logger = log;
// Ask the user if update is available
autoUpdater.on('update-downloaded', (event, info) => {
let message = app.getName() + ' ' + info.releaseName + ' is now available. It will be installed the next time you restart the application.';
if (info.releaseNotes) {
const splitNotes = info.releaseNotes.split(/[^\r]\n/);
message += '\n\nRelease notes:\n';
splitNotes.forEach(notes => {
message += notes + '\n\n';
});
}
// Ask user to update the app
dialog.showMessageBox({
type: 'question',
buttons: ['Install and Relaunch', 'Later'],
defaultId: 0,
message: 'A new version of ' + app.getName() + ' has been downloaded',
detail: message
}, response => {
if (response === 0) {
setTimeout(() => autoUpdater.quitAndInstall(), 1);
}
});
});
// init for updates
autoUpdater.checkForUpdates();
}
exports = module.exports = {
appUpdater
}; Q1. Does it look good, anything to add/remove? |
#1174 (comment) Will published in 1 hour. |
electron-updater 1.4.0 published.
Please updated docs https://github.com/electron-userland/electron-builder/wiki/Auto-Update
So, you need to change to |
@develar Please see the updated autoupdater function #1171 (comment) 😄 |
@akashnimare I prefer to notify using system notifications. To avoid annoying message boxes. e.g. just https://github.com/develar/onshape-desktop-shell/blob/master/src/AppUpdater.ts#L22 But if you prefer this way — code looks good. Except that you should get releaseName/releaseNotes from |
@develar hmm, yeah I think you're right. Dialog message boxes look kind of annoying. I'll change it system notifications :) |
|
aaaaahkk, I'm stupid. Thanks 😆 |
Just a suggestion - we could change it to -
|
11.15.1
Windows 7
Here is the full log -
Here is my package.json
I'm sure it's caused by squirrel.windows and I have gone through similar issues (Squirrel/Squirrel.Windows#710, Squirrel/Squirrel.Windows#698
) as well, still no luck.
The text was updated successfully, but these errors were encountered: