pokerogue
pokerogue copied to clipboard
Implement type immunity removal moves
The scope of this PR is to implement the following moves:
- Odor Sleuth
- Miracle Eye
- Foresight
The goal is to make the evasiveness reset and type removal work.
Included a demo for odor sleuth, checking to make sure everything works, it would be great if the author does something similar for the rest that they implemented
https://github.com/pagefaultgames/pokerogue/assets/39450497/2ecfe207-de48-45ff-9c60-5778effc758b
@DustinLin Thanks for the demo and the review. I cleaned up the nits.
About the recording - I might do it during the weekend. The other moves follow the exactly same logic as Odor Sleuth though. If one works, all should.
EDIT: Never mind - the imports got scrambled during the merge. Will fix that tomorrow.
Adding videos for the other two moves.
https://github.com/pagefaultgames/pokerogue/assets/60473007/d99d7b27-45bc-4037-a1a4-2c70bbcacfa1
https://github.com/pagefaultgames/pokerogue/assets/60473007/e64fc237-56b1-48bf-b6b2-7d3508a80cf1
Looks like there are conflicts on this currently
Looking to get JSDoc documentation as shown here: Code Comment FAQ
Added the docs, is it better now?
Can we get a video of the same scenarios from the original comment to show that evasion no longer gets reset?
https://github.com/pagefaultgames/pokerogue/assets/60473007/27895635-c127-4b5c-a3a1-c6ad81c128a1
Here is a video of Odor Sleuth. Are the other two videos necessary? They all follow the same logic. Either all work or none.
For the most part, these moves are working correctly, evasion and types are being ignored.
But going over some edge cases, mainly Wonder Guard and Mind's Eye (basically same functionality as Odor/Foresight):
Mind's eye allows hitting through Wonder Guard with a fighting type move:
https://github.com/pagefaultgames/pokerogue/assets/171270136/87187c27-0048-4280-99de-a8903c898393
However, after using foresight i can't hit it anymore:
https://github.com/pagefaultgames/pokerogue/assets/171270136/480cf340-bce5-4006-abd2-d26e02e13e4b
And just in case here's Wonder Guard vs just Foresight:
https://github.com/pagefaultgames/pokerogue/assets/171270136/8ada1340-548e-4f2e-88a5-b4ffe8b9c496
Unsure how this interaction is supposed to go, but in theory both Mind's Eye and Foresight should have the same result vs wonder guard. And why does Odor/Foresight make Mind's Eye stop working.
Will look into the interaction with Wonder Guard soon-ish.
https://github.com/pagefaultgames/pokerogue/assets/60473007/d9bf1c23-e01a-490d-a607-24123c9abd5f
Fixed the interaction. I thought I was being smart with the way I initially implemented it. Oh boy, was I wrong.
I'm sorry this was left un-reviewed for 3 weeks, if you update the merge conflicts ping me in discord and I can review it
Imo there should be some unit tests as well.
make sure to address the conflicts and the comments
If you can fix the merge conflicts and add some unit tests I think this looks fine
Closed in favor of #3379