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

Split reports by core project #10

Merged
merged 22 commits into from
Jul 2, 2024
Merged

Split reports by core project #10

merged 22 commits into from
Jul 2, 2024

Conversation

vincerubinetti
Copy link
Collaborator

@vincerubinetti vincerubinetti commented Jun 18, 2024

Changes roughly in order of diff:

  • add test workflow. simplify and make use of root-level run script.
  • elaborate readmes
  • add logo and favicon
  • update package versions
  • add vue router
  • add some icons
  • add some basic components, including table with tanstack table
  • generate more data json files for app
  • add dedicated page for each core project with basic info and table of publication details
  • add listing of core projects
  • add some global/total stats to homepage
  • style tweaks, including more print styles
  • lift certain important paths/vars up into .env file
  • make pipeline more modular and flexible. split up into different steps/files/folders. more consistent process. env vars for important switches. open preview for each step.
  • add root-level run script to conveniently run commands

Copy link

netlify bot commented Jun 18, 2024

Deploy Preview for cfde-eval ready!

Name Link
🔨 Latest commit 6427213
🔍 Latest deploy log https://app.netlify.com/sites/cfde-eval/deploys/6684044560dd0100073e99ff
😎 Deploy Preview https://deploy-preview-10--cfde-eval.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@cgreene
Copy link
Collaborator

cgreene commented Jun 27, 2024

This looks to move totally to project-level reporting. Will we lose the program-level dashboard with this? I think we must do the project-level reporting, but it was nice having the quick snapshot at the program level.

@vincerubinetti
Copy link
Collaborator Author

There are some CFDE-total stats on the home page. Before this PR I had some pie charts like the breakdown of project number prefixes. I got rid of those here since it was just placeholder visualization and I figured that wasn't useful information. I can add it back in or any other program level info that you think is useful.

@vincerubinetti
Copy link
Collaborator Author

Hopefully I'm understanding "program level" right. Perhaps it is actually "opportunity numbers". Either way, I think things are set up to pretty quickly add whatever data I have access to at any level you want in this PR.

Copy link
Member

@seandavi seandavi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the information content associated with this approach. I'd suggest adding some semblance of an "award title" to the project page just to make it easier to find projects of interest. You might also add the "award mechanism code" (OTA, R03, etc.) as a column on the first page as well. Our main audience will be NIH folks at this point, so giving those couple of details as top level could help them identify their respective programs/projects.

@vincerubinetti vincerubinetti merged commit 1265352 into main Jul 2, 2024
2 checks passed
@vincerubinetti vincerubinetti deleted the multi-reports branch July 2, 2024 19:11
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