Skip to content

JetBrains: Add error messages for failed actions

Warren Gifford requested to merge dv/add-error-messages-for-failed-actions into main

Created by: vdavid

One of our customers has been having issues with both the internal and external browsers. We're not sure whether we can solve all their problems as they might occur because of their environment. So at least we're trying to handle these edge cases gracefully to help all our users the best we can, and also to be able to get more meaningful error reports from them in the future.

This is what's displayed in each case when an external browser cannot be opened:

image

This is what's displayed in the rare cases when JCEF is not supported altogether:

image

Notes:

  • Also included a few trivial refactors: some class renames and moving a folder. It's many changes on the file level, so I've done them in commits separate from the rest of the changes. The refators would've clashed with the rest of the changes so I didn't want to create two different PRs, in order to move quickly.
  • Also bumped the version to alpha-5

Test plan

  • I've tested the notifications with an artificially broken environment when I created the error messages
  • I've tested the overall functionality after the refactors

App preview:

Check out the client app preview documentation to learn more.

Merge request reports

Loading