Project

General

Profile

Actions

Bug #16472

closed

Shows error snackbar even when the code will handle the error.

Added by Lucas Di Pentima almost 4 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Target version:
Story points:
-
Release relationship:
Auto

Description

We have added in the past a service layer feature that makes any request error to show up on the UI. This is useful as a catch-all measure to see hidden issues but sometimes workbench2 makes requests that have the potential to fail and the code even handles those failures, but the UI snackbar is displayed the same, making for a confusing UX.

There's some WIP in the branch 16118-service-layer-snackbar-fix, adding an optional parameter to the service layer methods so that on those cases when the error is handled, the snackbar can be avoided, but it's not working properly yet.
This branch comes from working on #16118, because the snackbar was originally causing issues with its tests, but as that was fixed in some other way, it is now on its own ticket.


Subtasks 1 (0 open1 closed)

Task #16484: Review 16472-service-layer-snackbar-fixResolvedPeter Amstutz06/23/2020Actions

Related issues

Related to Arvados - Bug #16118: Offers editing actions on read-only collectionsResolvedLucas Di Pentima02/28/2020Actions
Actions #1

Updated by Lucas Di Pentima almost 4 years ago

  • Related to Bug #16118: Offers editing actions on read-only collections added
Actions #2

Updated by Lucas Di Pentima almost 4 years ago

  • Target version changed from 2020-06-03 Sprint to 2020-06-17 Sprint
Actions #3

Updated by Lucas Di Pentima almost 4 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Lucas Di Pentima almost 4 years ago

  • Target version changed from 2020-06-17 Sprint to 2020-07-01 Sprint
Actions #5

Updated by Lucas Di Pentima almost 4 years ago

Rebased & renamed the mentioned branch on arvados-workbench2|61e9b81 - branch 16472-service-layer-snackbar-fix
Test run: developer-tests-workbench2: #52

  • Adds boolean parameter to the service layer so that on those cases where the code handles possible request failures, the "error snackbar" and console message are not displayed.
  • Uses the parameter to avoid showing an error when using the "ancestors service", as it was already handling errors. This makes accessing projects on "shared with me" to not show an error snackbars stream.
Actions #6

Updated by Peter Amstutz almost 4 years ago

Lucas Di Pentima wrote:

Rebased & renamed the mentioned branch on arvados-workbench2|61e9b81 - branch 16472-service-layer-snackbar-fix
Test run: developer-tests-workbench2: #52

  • Adds boolean parameter to the service layer so that on those cases where the code handles possible request failures, the "error snackbar" and console message are not displayed.
  • Uses the parameter to avoid showing an error when using the "ancestors service", as it was already handling errors. This makes accessing projects on "shared with me" to not show an error snackbars stream.

This LGTM.

Actions #7

Updated by Anonymous almost 4 years ago

  • Status changed from In Progress to Resolved
Actions #8

Updated by Lucas Di Pentima over 3 years ago

  • Release changed from 20 to 25
Actions

Also available in: Atom PDF