Skip to content

Commit

Permalink
Revert "fix: Await full sync stop before closing the app (#2300)" (#2302
Browse files Browse the repository at this point in the history
)

This reverts commit 0901d11.

By awaiting the full sync stop, we prevent quitting the app during a
synchronisation or a watcher run while these can take a very long time
(especially the first scan of local and remote changes).

We'll bring back this behavior once we have a way to cancel the
on-going sync and watch runs.
  • Loading branch information
taratatach authored May 9, 2023
1 parent 565cfc5 commit 1c8a1f3
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions gui/js/tray.window.js
Original file line number Diff line number Diff line change
Expand Up @@ -275,13 +275,9 @@ module.exports = class TrayWM extends WindowManager {
},
'auto-launcher': (event /*: ElectronEvent */, enabled /*: boolean */) =>
autoLaunch.setEnabled(enabled),
'close-app': async () => {
try {
await this.desktop.stopSync()
await this.app.quit()
} catch (err) {
log.error({ err, sentry: true }, 'error while quitting client')
}
'close-app': () => {
this.desktop.stopSync()
this.app.quit()
},
'unlink-cozy': () => {
if (!this.desktop.config.isValid()) {
Expand Down

0 comments on commit 1c8a1f3

Please sign in to comment.