Arvados

Changes at arvados-workbench2|81e046c27b73760acc5bffd51019516a2bbad94c
Tests developer-tests-workbench2: #461

I created a new renderer function to provide username from UUID and a helper function to reduce code duplication. I had to squeeze some of the other columns to make space.

Some comments:

• I think the new column is too narrow, possible solutions would be:
  • Make the right-hand side panel width-adjustable like the left-hand side panel
  • Add the user's name/uuid data as a second row spanning all columns of the same version's row
• The font used for the name/uuid is a little bigger than the font of the rest of the version browser (14 vs 12 px)
• I think the new column's name should say something like "Modified by" instead of just "User", so it's not confused with its owner (please check with Peter just in case)
• Could you update the cypress test to check for the new data being displayed on the collection browser? Bonus points if the test confirms that a version created by a user other than the owner is correctly shown on the UI. (for example, making some change with the admin user on a regular user owned collection)

Updated 6d6ea46757e93939c12d026faf9fd3c4d7b83122
Tests developer-tests-workbench2: #464

I addressed all the points you raised - there's one potential problem that the "Admin User" shows up as a UUID, I'm not sure if this is because of how the permissions between the adminUser and activeUser in cypress that prevents the code in renderers from getting the user details or whether there is some omission in fetching user details that needs to be fixed. In theory I think renderers.tsx:L432 should be fetching user details when available.

Stephen Smith wrote:

I addressed all the points you raised - there's one potential problem that the "Admin User" shows up as a UUID, I'm not sure if this is because of how the permissions between the adminUser and activeUser in cypress that prevents the code in renderers from getting the user details or whether there is some omission in fetching user details that needs to be fixed. In theory I think renderers.tsx:L432 should be fetching user details when available.

I think this works great, the issue with the admin user's full name I believe is what you described: the normal user doesn't have read access to it. Looks good to merge, thanks!