-
-
Notifications
You must be signed in to change notification settings - Fork 968
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
Sprite for Galarian Moltres is the one for Galarion Zigzagoon #688
Comments
Ugh, thanks for the info. Would you be able to propose a fix? |
I would gladly help but I don't know how to fix that and I don't know Python. |
Argh!!!! All the galar varieties mismatch! We have a problem. I think I'll revert to a previous version. Let's keep analyzing using the staging version. https://staging.pokeapi.co/api/v2/pokemon/10170/ |
So last PR (PokeAPI/pokedex#86) messed up some IDs, I think the ones above 10000. But I didn't fully analyze the issue |
Ok |
Right, we shifted the galarian id's but we didn't rename the sprite images since it's tied with their id. In the |
So in pokemon_forms.csv everything between 10158 and 10317 needs to be renamed ? |
Yeah, this should be a one time thing since we're syncing up with veekun. I'll try to rename the existing sprites to the new id's with a script and make a PR. |
Ok |
Ok, it seems to work, can you confirm @thibaudbrault ? huge thanks to @C-Garza |
Right now I can't but I'll look tomorrow |
I just checked an it works; Thx for fixing it :) |
What is said in the title the sprites for galarian moltres are the ones for galarian zigzagoon
The text was updated successfully, but these errors were encountered: