Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Note and Doc Link for Manifest Download #99

Open
b-f-chan opened this issue Mar 24, 2021 · 5 comments
Open

Add Note and Doc Link for Manifest Download #99

b-f-chan opened this issue Mar 24, 2021 · 5 comments
Assignees
Labels

Comments

@b-f-chan
Copy link

Add the note and documentation link in the File Manifest download button per original mockups:

image.png

The instructions link should go to https://overture.bio/documentation/score/user-guide/download

@b-f-chan
Copy link
Author

b-f-chan commented Apr 1, 2021

@caravinci will cover as part of Arranger re-write after chatting with him, because he wants to do it properly with the refactor

@justincorrigible
Copy link
Member

In this PR, I've added something similar to the design by @kcullion (as customising arranger to look fully like the mock, would imply a larger set of changes).
Note: using the link described in the ticket may need to be modified, as it currently leads to a 404.

@kcullion
Copy link

kcullion commented Apr 6, 2021

In this PR, I've added something similar to the design by @kcullion (as customising arranger to look fully like the mock, would imply a larger set of changes).
Note: using the link described in the ticket may need to be modified, as it currently leads to a 404.

sounds good to me.
yeah the link would probably go to this section (but @b-f-chan I don't see a section on that page for user to download data using a file manifest) https://www.overture.bio/documentation/dms/user-guide/data-portal/

@b-f-chan
Copy link
Author

Confirming new/final link will be:
https://www.overture.bio/documentation/score/user-guide/download

Because using the manifest to download is actually a score client function, not DMS UI.

@justincorrigible
Copy link
Member

justincorrigible commented Apr 30, 2021

Updating PR to include the URL.


Edit: as it turns out, that was the link I had added in the PR. It's good for Review, then.

@b-f-chan b-f-chan removed this from the Canarie Sprint 17 milestone May 2, 2021
This was referenced Dec 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants