update-electron-app icon indicating copy to clipboard operation
update-electron-app copied to clipboard

Add option for including release notes in update dialog

Open davej opened this issue 6 years ago • 9 comments

Just wondering if showing the release notes when prompting the user to restart is on the roadmap? Release notes would be taken from the github release.

Here's a potential mock-up with a toggle to show/hide:

release-notes

An option could toggle whether to show them or not:

require('update-electron-app')({
  repo: 'ummoji/ummoji-desktop',
  releaseNotes: true
})

Even if release notes are turned on they would only be shown if there are actually release notes associated with the release. The release notes would probably need to be contained with a maximum height and overflow content should scroll because otherwise the height of the dialog may be too large. Perhaps this should be configurable or perhaps not.

davej avatar Apr 22 '18 11:04 davej

@davej I'm in favor of this. Do you want to give it a try?

zeke avatar Apr 23 '18 16:04 zeke

@zeke Ok, I could have a go at the JavaScript stuff. I think the collapsible release notes would require changes to Electron core and some C code though. Any tips on that? Or perhaps there's a way to approach it without touching Electron core?

davej avatar Apr 23 '18 16:04 davej

Windows also has the concept of collapsible text in a dialog. Perhaps this would be a useful addition to Electron core? It's much nicer UX than plonking all the release notes inline on a dialog. Is it worth me opening an issue on Electron Core?

image

image

davej avatar Apr 23 '18 17:04 davej

Or perhaps there's a way to approach it without touching Electron core?

Yeah that would be ideal. Since this is designed for use with GitHub Release, we could also offer the user a link to the release page... 🤔

@MarshallOfSound @zcbenz is there a way to know at app startup that the app itself was recently updated? Don't see a hook for that in the autoUpdater docs.

zeke avatar Apr 23 '18 20:04 zeke

@MarshallOfSound @zcbenz is there a way to know at app startup that the app itself was recently updated? Don't see a hook for that in the autoUpdater docs.

The native auto-update frameworks used in Electron do not provide the functionality, it is usually implemented in apps by storing current version in a configuration file.

zcbenz avatar Apr 23 '18 23:04 zcbenz

https://github.com/electron/electron/issues/12437 would make this easier to implement.

sindresorhus avatar May 02 '18 06:05 sindresorhus

cảm ơn rất nhiều

quanganh1906 avatar Jun 19 '19 01:06 quanganh1906

Ref: https://github.com/electron/electron/pull/23692

Not exactly the toggle, but definitely and improved experience

erickzhao avatar Sep 30 '20 23:09 erickzhao

No more updates?

jhaemin avatar Aug 08 '24 15:08 jhaemin