-
Notifications
You must be signed in to change notification settings - Fork 318
Enhancements to Galaxy server instances on Use Page #3376
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
Enhancements to Galaxy server instances on Use Page #3376
Conversation
…pes' (public servers, VMs, etc) * Direct use of Resource Type querystring loads correct data subset * De-emphasized url path in table cells * Added column sort on parameter-based column value with visual indicator in column header * Code separation of functions: table sorting and table routing * Added Tier column with sorting
* Add new geographic Region column * Updated routing querystring name to align with YAML standards * Added Region column with sorting
6249e53 to
a428b95
Compare
…into instances-routing-structure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but could you add some info/tooltip to describe "Tier" because I've no idea what it means in this context?
@neoformit Yes - this makes sense. Even after this ticket is merged/deployed, the Tiers column will not yet appear until it is uncommented out and until the data in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These look like good additions!
…into instances-routing-structure
@neoformit, also thanks for the good call-out. The Tiers should just be further refinement of the Galaxy membership process listed on this UseGalaxy Servers page's Membership section. So we will likely need to update this section and possibly communicate it internally. |
Part 1 of 2 - basic ui/ux feature improvements to Galaxy Instances table
This PR addresses conservative changes to the Galaxy Instances table Next PR for Galaxy Instances table will address more advance feature improvements (server availability, geolocation) to the table as well as the "Use" page overall.
To code/content reviewers: please note your comments with Priority = high / medium / low
Targeted for merge Wed or Thu