Details
-
Type:
Task
-
Status: Closed (View Workflow)
-
Priority:
Blocker
-
Resolution: Done
-
Affects Version/s: None
-
Fix Version/s: None
-
Labels:None
-
Story Points:2
-
Sprint:Fall 7 : 11 Nov to 22 Nov, Fall 8 : 25 Nov to 6 Dec
Description
Situation: We need an intermediate page (bioviz.org/cyverse) that informs the user what BioViz-CyVerse is and has a button linking to the BioViz-CyVerse page (currently using nowlan.bioviz.org).
Task: Use the already existing https://test.bioviz.org/cyverse.html page as a template for bioviz.org/cyverse, but change the button to link to nowlan.bioviz.org. Add a CyVerse Data button in the navbar that links to bioviz.org/cyverse Then issue a pull request from your fork to the master branch.
Attachments
Activity
| Field | Original Value | New Value |
|---|---|---|
| Epic Link |
|
| Assignee | Philip Badzuh [ pbadzuh ] |
| Status | To-Do [ 10305 ] | In Progress [ 3 ] |
| Sprint | Fall 7 : 11 Nov to 22 Nov [ 78 ] | Fall 7 : 11 Nov to 22 Nov, Fall 8 : 25 Nov to 6 Dec [ 78, 79 ] |
| Rank | Ranked higher |
I have a request:
Can you comment the "CyVerse Data link" in menu.html so that it will not be visible by default?
When we deploy for testing, we can just make a local edit to remove the comment.
And when we release, we can do the same, but commit the change to master.
cc: Nowlan Freese
Sure, no problem. The change should be reflected in the same link.
| Assignee | Philip Badzuh [ pbadzuh ] |
| Status | In Progress [ 3 ] | Needs 1st Level Review [ 10005 ] |
Tested on Philip's EC2.
CyVerse Data link is hidden.
CyVerse page links to nowlan.bioviz.org.
Recommend pull request.
| Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
| Status | First Level Review in Progress [ 10301 ] | Ready for Pull Request [ 10304 ] |
| Assignee | Philip Badzuh [ pbadzuh ] |
Philip Badzuh - please submit PR from your branch.
| Comment |
[ I have submitted a PR here:
https://bitbucket.org/lorainelab/appstore/pull-requests/193/igbf-2187-fix-sorting-by-download-count-on/diff ] |
I have submitted a PR here:
https://bitbucket.org/lorainelab/bioviz/pull-requests/34/igbf-2155-add-cyverse-page-and-link-to-it/diff
| Status | Ready for Pull Request [ 10304 ] | Pull Request Submitted [ 10101 ] |
| Assignee | Philip Badzuh [ pbadzuh ] |
| Status | Pull Request Submitted [ 10101 ] | Reviewing Pull Request [ 10303 ] |
| Status | Reviewing Pull Request [ 10303 ] | Merged Needs Testing [ 10002 ] |
To test:
- Visit: https://bioviz.org/cyverse.html
- Confirm that you can log in to your CyVerse account
- Confirm that the redirect URI goes somewhere you can access CyVerse data.
I followed the above workflow and was able to see data files for IGB in the google-drive inspired UI.
| Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
| Resolution | Done [ 10000 ] | |
| Status | Post-merge Testing In Progress [ 10003 ] | Closed [ 6 ] |
| Assignee | Philip Badzuh [ pbadzuh ] |
I have added the changes in my branch here:
https://bitbucket.org/pbadzuh/bioviz_pbdev/branch/IGBF-2155#diff