Arvados

• arv-put finally puts the file in a reasonable place. Very exciting!
• The "Move project" and "Delete project" buttons are visible and enabled on the home project page. They should be disabled or invisible.
• The breadcrumbs bar is the same if you are on the Home project or the Dashboard. It would be nice if there were some visual cue to indicate which of those pages you are actually on.

FAIL: test_KeepBasicRWTest (test_keep_client.KeepPermissionTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/peter/work/arvados/sdk/python/tests/test_keep_client.py", line 119, in test_KeepBasicRWTest
    unsigned_bar_locator)
AssertionError: NotFoundError not raised

Peter Amstutz wrote:

• The "Move project" and "Delete project" buttons are visible and enabled on the home project page. They should be disabled or invisible.

Right, fixed (by removing). Also removed "Sharing" tab, thinking sharing your entire account is something we should deliberately not provide an easy method for. (Is this reasonable?)

• The breadcrumbs bar is the same if you are on the Home project or the Dashboard. It would be nice if there were some visual cue to indicate which of those pages you are actually on.

Fixed. Also added "Home" to the "My projects" panel on Dashboard.

[...]

Haven't been able to reproduce that failed test.

Now at 1076b57

• Move/Delete buttons are fixed.
• Breadcrumbs are fixed
• I tested the "Add data" and "New subproject" buttons and they seem to work as expected
• I went to "run a pipeline" and got this box:

Hey. This pipeline instance belongs to your account, but it's not in any of your projects. If you want it to be easy to find in the future, you should move it to a project.

• We talked about having links (except for name and permission) show up under "Other objects", are we going to do that?

Peter Amstutz wrote:

Hey. This pipeline instance belongs to your account, but it's not in any of your projects. If you want it to be easy to find in the future, you should move it to a project.

I think we can safely get rid of this warning now.

Indeed. Removed.

• We talked about having links (except for name and permission) show up under "Other objects", are we going to do that?

Possibly, but surely not as part of this story.

Review feed back for the 3499-doc branch:

1. api-tokens.html.textile.liquid

In the sentence “Click on the link with your email address in the upper right corner to access the user settings menu” — We call our “gear” menu as “settings menu”. It is better to say “My account” menu, since we now have that name in this drop down menu.

2. tutorial-keep.html.textile.liquid

- “To move the collection to a specific project” — how about “To move the collection to a different project”?

- I think it would be nice if you add a picture of the “move selected” button

3. tutorial-pipeline-workbench.html.textile.liquid

- There are a couple of typos in the first paragraph (you did not update it, but since you are updating this page anyways :)

A “pipeline” . . . is sequence of - - it is better to say “is a sequence of” (missing the word “a”)

“small data set of paired-end reads from an sample” - - should be “from a sample” (used “an” instead of “a”)

- “You can return to the Dashboard by clicking on” - - how about “You can go to . . . ” or “You can access the Dashboard . . . “

- “On the My projects panel” - - I think “In the My projects panel” would be better

- “This will direct you to the page for the new project.” - - Can we say “This will create a new project and take you to the page for the new project”?

- “to the right of the SAM file download your results” - - “to the right of the SAM file to download your results” (missing the word “to”)

- “Click on <span class="btn btn-sm btn-primary" >Run <i class="fa fa-fw fa-play"></i></span>” - - This looks pretty odd without the word “ button” after the Run button.

- Can you please add the word button or drop down or whatever after all the span elements. I copied a few more occurrences below.

“Click on <span class="btn btn-sm btn-primary"><i class="fa fa-fw fa-gear"></i> Run a pipeline...</span>” - - how about adding the word “button” after this

“Click on All projects <span class="caret"></span>” - - how about adding “drop down” after this

“Under the Projects shared with me header, select <i class="fa fa-fw fa-share-alt"></i> Arvados Tutorial” - - how about adding the word “project” after this

Radhika,

All these suggestions are great. I've pushed a new commit to incorporate everything you suggested.

Radhika Chippada wrote:

- “This will direct you to the page for the new project.” - - Can we say “This will create a new project and take you to the page for the new project”?

I did use a different sentence here because I thought it sounded awkward to repeat "project" so often, but I hope the change follows the flow of your suggestion.

Ready for another look at 8402746. Thanks.

Brett, everything looks great now, but for the one concern below.

- In the newly added selection image "workbench-move-selected.png", I think it would be most helpful to the users if you were to include the tabs also. I think the user needs to see "Data collections" tab (and those that fit in within the dropdown area) to know right away where the Selection dropdown is located.

After that, everything LGTM. thanks.

Radhika Chippada wrote:

- In the newly added selection image "workbench-move-selected.png", I think it would be most helpful to the users if you were to include the tabs also. I think the user needs to see "Data collections" tab (and those that fit in within the dropdown area) to know right away where the Selection dropdown is located.

That sounds good too. The new commit is 35a3223 and merged with master. Thanks again.