Arvados

All items finished - Waiting for cypress fixes in 15768 to be merged.

developer-tests-workbench2: #1352

• All agreed upon points are implemented / addressed.
  • done
• Anything not implemented (discovered or discussed during work) has a follow-up story.
  • N/A
• Code is tested and passing, both automated and manual, what manual testing was done is described
  • Manually navigated from user to group several times
• Documentation has been updated.
  • N/A
• Behaves appropriately at the intended scale (describe intended scale).
  • N/A
• Considered backwards and forwards compatibility issues between client and server.
  • N/A
• Follows our coding standards and GUI style guidelines.
  • done

For the future it would be nice to split out formatting changes into a separate commit so it's easier to review the changes

Just one minor fix:

• One caveat with kind === group is that both user groups, projects, and virtual projects are "kind === group", so those are differentiated by the groupClass property which is GroupClass.ROLE for role (user) groups - you can see that property used on line 516 so you'll just need to include a check for that to make sure project resources use the regular navigateTo method.
• There's an unused getUserUuid import warning that we might as well remove while we're here

Once that's done I think it lgtm and you can merge