-
Notifications
You must be signed in to change notification settings - Fork 2
Agencies: Added agency logos #39
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
Conversation
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
| # "Department of Labor": "https://www.dol.gov/code.json", | ||
| # "Department of Transportation": "https://www.transportation.gov/code.json", | ||
| "DoT": "https://www.treasury.gov/code.json", | ||
| "TREASURY": "https://www.treasury.gov/code.json", |
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.
Updated agency names to match acronyms list in the original code-gov implementation: https://github.com/GSA/code-gov-data/blob/master/agency_metadata.json
IDEA: I think we should separate out this agency metadata to its own file like how it is done there. I can forsee the frontend also needing to use this data
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.
IDEA for later: Building on top of this, we should display the agency's full name in the agency cards for users that do not know the acronyms. We can use the separate agency metadata file to make this happen @DinneK
| # the commented out links are not valid or cant be found | ||
| agencies_links = { | ||
| "DoA": "https://usda.gov/code.json", | ||
| "USDA": "https://usda.gov/code.json", |
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.
Because I made this change, we'll need to rerun the scripts. I'll message you guys tmr for your help @sachin-panayil @IsaacMilarky
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.
Instead, decided to run the scripts using GitHub Action after merging this PR
DinneK
left a comment
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.
Pulled locally, LGTM!
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Signed-off-by: Natalia Luzuriaga <natalia.luzuriaga@cms.hhs.gov>
Agencies: Added agency logos
Problem
We would like to add agency logos to our agency cards. Currently, we have an emoji as a placeholder.
Solution
Using the images in
agency_logodirectory, added image to agency card.agency.codeshould match with agency logo title (example: usda -> usda.png)Result
Each agency card now has their logo shown at the top right!
Test Plan
Tests passed,
npm run dev