Project

General

Profile

Actions

Feature #16659

closed

Add a "Copy to Clipboard" Button for the token

Added by Daniel Kutyła over 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Target version:
Start date:
08/07/2020
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

Users setting up their shell for accessing the Arvados API always need to copy their tokens from the dialog. A little simplification could be to provide a button to copy the token snippet into clipboard.


Subtasks 1 (0 open1 closed)

Task #16674: Review 16659-copy-api-token-to-clipboardResolvedDaniel Kutyła08/07/2020

Actions

Related issues 1 (0 open1 closed)

Related to Arvados - Feature #16671: Copy-to-clipboard project uuidResolved

Actions
Actions #1

Updated by Daniel Kutyła over 4 years ago

  • Target version set to 2020-08-12 Sprint
Actions #2

Updated by Peter Amstutz over 4 years ago

Actions #4

Updated by Lucas Di Pentima over 4 years ago

Reviewing arvados-workbench2|2ee1db2

  • Please take note that some tests on Jenkins failed.
  • When I tried to start it with yarn start, I got an error message:
    Failed to compile.
    
    /Users/lucas/Devel/git/arvados-workbench2/src/views-components/current-token-dialog/current-token-dialog.tsx
    (56,37): Unnecessary semicolon
    
  • There're several unrelated changes on this commit, were those on purpose? Maybe there's some automatic tool making them on your IDE, just wanted point that out.
Actions #6

Updated by Lucas Di Pentima over 4 years ago

  • Status changed from New to In Progress
  • Assigned To set to Daniel Kutyła

Reviewing arvados-workbench2|27f1790

  • The env vars text box now show some lines indented instead of all being aligned to the left.
  • The ticket’s description mentions easing the users’ worflow when they try to access a shell node, but the button only copies the token and not the entire set of environment variables that the users need to paste on their shell session.
  • The test checks for a snackbar to appear instead of the main desired effect (data on clipboard). I’ve checked and it seems that a Clipboard API is in the works and already has some browser support, maybe it can be used to write a test that checks the correct data is being copied to the clipboard: https://developer.mozilla.org/en-US/docs/Web/API/Clipboard_API
  • If the entire text box is what is needed to be copied, the button's label could say "COPY TO CLIPBOARD" instead of "COPY API TOKEN"
Actions #7

Updated by Peter Amstutz over 4 years ago

  • Target version changed from 2020-08-12 Sprint to 2020-08-26 Sprint
Actions #8

Updated by Daniel Kutyła over 4 years ago

New version: https://dev.arvados.org/projects/arvados/repository/arvados-workbench2/revisions/bbcdcd7b7582b43ca4853e436df383ff9dda465a
Test run: developer-tests-workbench2: #75

Added copy of full token, made tests more realistic

Please check in git cli I do not know why but diff does not show any changes here

Actions #9

Updated by Lucas Di Pentima over 4 years ago

This LGTM, thanks!

Actions #10

Updated by Daniel Kutyła over 4 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100
Actions #11

Updated by Peter Amstutz about 4 years ago

  • Release set to 25
Actions

Also available in: Atom PDF