Skip to content

Conversation

@natalialuzuriaga
Copy link

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_logo directory, added image to agency card. agency.code should match with agency logo title (example: usda -> usda.png)

Result

Screenshot 2025-06-25 at 2 45 40 PM

Each agency card now has their logo shown at the top right!

Test Plan

Tests passed, npm run dev

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",
Copy link
Author

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

Copy link
Author

@natalialuzuriaga natalialuzuriaga Jun 25, 2025

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",
Copy link
Author

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

Copy link
Author

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

@natalialuzuriaga natalialuzuriaga self-assigned this Jun 25, 2025
@natalialuzuriaga natalialuzuriaga added the enhancement New feature or request label Jun 25, 2025
Copy link

@DinneK DinneK left a 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>
@natalialuzuriaga natalialuzuriaga merged commit 3d2667a into main Jul 1, 2025
6 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants