hyper icon indicating copy to clipboard operation
hyper copied to clipboard

Auto-update if all windows closed

Open leo opened this issue 8 years ago • 13 comments
trafficstars

IMO, we should just restart automatically after downloading an update if no Hyper window is open.

leo avatar Dec 31 '16 22:12 leo

This is really clever

rauchg avatar Dec 31 '16 22:12 rauchg

As long as there is a notice I'm fine with that.

einfallstoll avatar Jan 01 '17 12:01 einfallstoll

I wouldn't expect that to happen...

albinekb avatar Jan 04 '17 18:01 albinekb

From the electron auto-updater docs:

Note: autoUpdater.quitAndInstall() will close all application windows first and only emit before-quit event on app after that. This is different from the normal quit event sequence.

If there is no window open, shouldn't quitAndInstall() just restart it? I mean, I would assume that that's the expected behavior, isn't it working like that right now?

pablopunk avatar Jan 07 '17 05:01 pablopunk

I prefer the macOS way. Give user the ability to update when they want.

ppot avatar Jan 07 '17 23:01 ppot

I like the Chrome way. Always run the latest, the least you bother the user the better.

rauchg avatar Jan 08 '17 02:01 rauchg

I like the Chrome way. Always run the latest, the least you bother the user the better.

Thumbs up for that 🙌🏻

IMO for patches or even minor versions it should be automatic.

pablopunk avatar Jan 08 '17 02:01 pablopunk

Yep, in the case of no terminal sessions being open, for all cases I think

rauchg avatar Jan 08 '17 03:01 rauchg

That would be great. How does the updating process work currently? Is it manual? I'm on Windows 10

AlbertoElias avatar Jan 09 '17 11:01 AlbertoElias

@rauchg is this issue valid yet? I started using Hyper a few days ago and I would like to contribute.

jhonnymoreira avatar Jun 02 '17 03:06 jhonnymoreira

Yep, I'd say we're happy to accept a PR for it! 😊

leo avatar Jun 02 '17 14:06 leo

thanks, @leo! Expect a PR soon. [=

jhonnymoreira avatar Jun 02 '17 15:06 jhonnymoreira

https://github.com/zeit/hyper/pull/2515

Stanzilla avatar Jan 23 '18 14:01 Stanzilla