Fix incorrect encounters in Gen IV games#1601
Conversation
jemarq04
left a comment
There was a problem hiding this comment.
Good catch! Your changes are good, I just noticed an issue with the pre-existing pokemon IDs if you can fix that up. Thanks!
|
fixed |
jemarq04
left a comment
There was a problem hiding this comment.
Great, looks good. Please also sync it with the master branch - if you have some difficulties let me know, as I was also experiencing some issues with that using git. I've had to sync with GitHub lately.
|
Done |
|
I added another fix. There was no Giratina encounters at all in platinum, when there are 2 (one conditional on not having done the first). |
|
Thanks! |
|
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 updater script has finished its job and has now opened a Pull Request towards PokeAPI/api-data with the updated data. |
Change description
The commit ef10632 was not implemented correctly. It incorrectly put a bunch of SoulSilver encounters in the platinum game, leading to encounter confusion (platinum showed a lapras encounter in union cave, a location only in HG/SS for example).
It also added a incorrect "event arceus in party" to the HG lapras encounter. It added it to one-to-many encounters, not just the ones it meant to.
You can see in the original commit, encounters added for games 14 and 15, but 14 is platinum.
AI coding assistance disclosure
Claude helped me find the issue after I noticed the errant Lapras encounter
Contributor check list