Page MenuHomePhabricator

[desktop] Ignore exceptions about being offline
AbandonedPublic

Authored by michal on Feb 10 2023, 2:04 AM.
Tags
None
Referenced Files
F3515903: D6684.diff
Sun, Dec 22, 10:57 AM
Unknown Object (File)
Nov 22 2024, 12:54 PM
Unknown Object (File)
Nov 20 2024, 6:12 PM
Unknown Object (File)
Nov 20 2024, 6:12 PM
Unknown Object (File)
Nov 20 2024, 6:12 PM
Unknown Object (File)
Nov 20 2024, 6:12 PM
Unknown Object (File)
Nov 20 2024, 6:10 PM
Unknown Object (File)
Nov 16 2024, 9:43 PM
Subscribers

Details

Summary

ENG-2923
If we call autoUpdater.checkForUpdates() while we are offline there will be an exception thrown. Unfortunately we can't just try{}catch{} but we can catch all exceptions and swallow all exceptions with a message about being offline. We already notify the user about being offline with the DISCONNECTED bar. If it's some other exceptions we can call dialog.showErrorBox to get the previous behaviour.

Test Plan

Check if autoUpdater.checkForUpdates() is ignored correctly. Check if other exceptions are displayed correctly.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ashoat added inline comments.
desktop/src/main.js
296 ↗(On Diff #22399)

Can we keep the same text as before?

This revision is now accepted and ready to land.Feb 13 2023, 2:59 PM