Bug #17637
closed
"Not found" error when trashing project
100%
Description
It displays a spurious "not found" error when trashing a project. It appears to happen because the project is still considered selected after it has been trashed.
Steps to reproduce:
- Create a subproject
- Click on the subproject so it is displayed in the main panel
- Right-click on the subproject on the left tree view and select "move to trash"
- The project will be correctly trashed, but it will pop up a confusing dialog saying the project is not found
Updated by Peter Amstutz over 3 years ago
- Blocks Story #17512: Release Arvados 2.2 added
Updated by Lucas Di Pentima over 3 years ago
- Status changed from New to In Progress
Updated by Lucas Di Pentima over 3 years ago
Updates at arvados-workbench2|55b961b - branch 17637-not-found-after-trash
Test run: developer-tests-workbench2: #415
- Fixes "not found" error message/dialog after trashing a project.
- Removes superfluous snackbar "About to do X..." messages when trashing groups and collections.
- Fixes error messages when untrashing collections and projects.
- Navigates to the recently untrashed project.
- Updates the projects tree on the left side panel after trashing a project.
- Adds cypress test.
Updated by Peter Amstutz over 3 years ago
Lucas Di Pentima wrote:
Updates at arvados-workbench2|55b961b - branch
17637-not-found-after-trash
Test run: developer-tests-workbench2: #415
- Fixes "not found" error message/dialog after trashing a project.
I found an edge case:
- Create a project
- Create a subproject
- Select the subproject
- Right click on the parent project and trash it
- You still get the "not found" error and the project tree gets confused
- Removes superfluous snackbar "About to do X..." messages when trashing groups and collections.
- Fixes error messages when untrashing collections and projects.
- Navigates to the recently untrashed project.
- Updates the projects tree on the left side panel after trashing a project.
- Adds cypress test.
Updated by Lucas Di Pentima over 3 years ago
Peter Amstutz wrote:
I found an edge case:
Good catch! I've simplified and fixed the issue, and in the process fixed another one that slipped through: the side panel tree wasn't being updated on some occasions.
Updates at arvados-workbench2|f6d52fc8
Test run: developer-tests-workbench2: #416
Updated by Peter Amstutz over 3 years ago
Lucas Di Pentima wrote:
Peter Amstutz wrote:
I found an edge case:
Good catch! I've simplified and fixed the issue, and in the process fixed another one that slipped through: the side panel tree wasn't being updated on some occasions.
Updates at arvados-workbench2|f6d52fc8
Test run: developer-tests-workbench2: #416
This LGTM, thanks!
Updated by Lucas Di Pentima over 3 years ago
- Status changed from In Progress to Resolved
Applied in changeset arvados-workbench2|976f4f2798b1704a864f8aa5d15e997ed1bf313f.