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

Dashboard status indicator #2500

Closed
wants to merge 3 commits into from

Conversation

erik-whiting
Copy link
Contributor

Working towards #2499

@matentzn can you take a quick look at my first pass? Here's basically what I'm doing:

  1. Parse the HTML of the OBO dashboard to find the pass/fail status of a specific ontology. (I know you said to translate the YML to JSON but as I mentioned, JavaScript doesn't work well with YML so translating it to JSON would present the same problem I had with just parsing the YML file, only with an extra step).
  2. Use the parsed data to set a status indicator and success sort order value.
  3. Use those values in the sortable table

With the current code, it looks like this:

image

I don't actually want to show the "success sort order" column, it will be hidden eventually. My challenge right now is setting that to be the default sort value, but I cannot figure out how to do that since we are using a custom data table, I don't think there's a "default column to sort by" function. Do you have any tips for getting me started on that?

Also, let me know if the color doesn't work.

@erik-whiting erik-whiting changed the title Erik/2499 dashboard status indicator Dashboard status indicator Dec 30, 2023
@matentzn
Copy link
Contributor

matentzn commented Jan 2, 2024

@erik-whiting Happy New Year, and thanks a ton for getting this going.

Some thoughts:

@matentzn
Copy link
Contributor

matentzn commented Mar 9, 2024

@jsstevenson this is the draft PR, you can use or not as much of this as you like!

@jsstevenson
Copy link
Contributor

continued in #2555

@anitacaron anitacaron deleted the erik/2499-dashboard-status-indicator branch November 9, 2024 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants