-
-
Notifications
You must be signed in to change notification settings - Fork 955
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
Enhancement: Updated Sprite URLs returned by API #727
Conversation
Previously the sprite endpoint returned the relative path of the image with a /media/ prefix which was not useful for API consumer. Now it returns a URL pointing to the image present in github.com/PokeApi/Sprites. Signed-off-by: Aniket Chakraborty <[email protected]>
Signed-off-by: Aniket Chakraborty <[email protected]>
Github isn't a CDN, whomever is consuming the API should be hosting the sprites themselves. |
Agreed. But that's when we are trying to scale the project. A lot of the API consumers are people like me making small side projects as a hobby. Besides, the |
|
Cool, it works! Thanks for the PR! |
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. |
Previously the sprite endpoint returned the relative path of
the image with a
/media/
prefix which was not useful for API consumer.Now it returns a URL pointing to the image present in github.com/PokeAPI/Sprites.
Signed-off-by: Aniket Chakraborty [email protected]