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

Missing sprites #81

Merged
merged 18 commits into from
Oct 17, 2022
Merged

Missing sprites #81

merged 18 commits into from
Oct 17, 2022

Conversation

devmike123
Copy link
Collaborator

No description provided.

@devmike123
Copy link
Collaborator Author

@Naramsim pr ready for merge

@Naramsim Naramsim merged commit c0e1d84 into PokeAPI:master Oct 17, 2022
@Naramsim
Copy link
Member

Thanks!

@devmike123 devmike123 deleted the missing-sprites branch October 31, 2022 03:30
@joseph-bayer
Copy link
Contributor

Hey @devmike123 and @Naramsim!

This pr provided a huge amount of missing art - when do the changes get reflected on the API?

I'm asking because I'm not sure if it's an automatic process to update the API or something that needs to be done manually. If it has to be done manually, I'd love to help!

Thanks,

-Joe

@devmike123
Copy link
Collaborator Author

Hey @devmike123 and @Naramsim!

This pr provided a huge amount of missing art - when do the changes get reflected on the API?

I'm asking because I'm not sure if it's an automatic process to update the API or something that needs to be done manually. If it has to be done manually, I'd love to help!

Thanks,

-Joe

It's automatic.

@joseph-bayer
Copy link
Contributor

@devmike123 - cool! Does a pipeline need to be ran? I don’t see these images when calling the current API. For example, the payload for Eternal Floette doesn’t have the official art url populated: https://pokeapi.co/api/v2/pokemon/floette-eternal

@C-Garza
Copy link
Member

C-Garza commented Nov 13, 2022

The sprites repo lives as a submodule in pokeapi here https://github.com/PokeAPI/pokeapi/tree/master/data/v2. The commit it's pointing to probably has to be updated.

@joseph-bayer
Copy link
Contributor

The sprites repo lives as a submodule in pokeapi here https://github.com/PokeAPI/pokeapi/tree/master/data/v2. The commit it's pointing to probably has to be updated.

Ah that makes sense! If the timing is right to update that, I can create a pull request pointing to the newest commit in the sprites repo (preferably after this pull request is merged).

@devmike123
Copy link
Collaborator Author

The sprites repo lives as a submodule in pokeapi here https://github.com/PokeAPI/pokeapi/tree/master/data/v2. The commit it's pointing to probably has to be updated.

Ah that makes sense! If the timing is right to update that, I can create a pull request pointing to the newest commit in the sprites repo (preferably after this pull request is merged).

With pleasure

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