Skip to content

feat: Update type icons to include logo (ref: sprites/pull/199)#1443

Merged
Naramsim merged 3 commits intoPokeAPI:masterfrom
FallenDeity:feat/small-icons
Mar 16, 2026
Merged

feat: Update type icons to include logo (ref: sprites/pull/199)#1443
Naramsim merged 3 commits intoPokeAPI:masterfrom
FallenDeity:feat/small-icons

Conversation

@FallenDeity
Copy link
Contributor

Adds a new field logo for type icons which references the small icons added in PokeAPI/sprites#199

data/v2/build.py Outdated
f"types/{generation}/{game}/{info[0]}.png"
)
),
"logo": file_path_or_none(
Copy link
Member

Choose a reason for hiding this comment

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

Is it better logo , small or symbol_icon? On bulbapedia there's the term symbol icon

Copy link
Contributor

Choose a reason for hiding this comment

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

i prefer icon

Copy link
Contributor Author

Choose a reason for hiding this comment

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

lets go with symbol_icon? matching bulbapedia is cool

Copy link
Member

Choose a reason for hiding this comment

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

ok, I also think that matching is better

Copy link
Member

@Naramsim Naramsim left a comment

Choose a reason for hiding this comment

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

Which naming convention is better?

@Naramsim
Copy link
Member

hi @FallenDeity can you run make format? black is complaining

@Naramsim Naramsim merged commit 19222e7 into PokeAPI:master Mar 16, 2026
9 checks passed
@pokeapi-machine-user
Copy link

A PokeAPI/api-data refresh has started. In ~45 minutes the staging branch of PokeAPI/api-data will be pushed with the new generated data.

The staging branch will be deployed in our staging environment and the entire API will be ready to review.

A Pull Request (master<-staging) will be also created at PokeAPI/api-data and assigned to the PokeAPI Core team to be reviewed. If approved and merged new data will soon be available worldwide at pokeapi.co.

@pokeapi-machine-user
Copy link

The updater script has finished its job and has now opened a Pull Request towards PokeAPI/api-data with the updated data.

The Pull Request can be seen deployed in our staging environment when CircleCI deploy will be finished (check the start time of the last build).

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.

4 participants