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

[Bug] Fix Post-Summon abilities not applying after initially accepting a switch prompt #4908

Open
wants to merge 2 commits into
base: beta
Choose a base branch
from

Conversation

innerthunder
Copy link
Collaborator

What are the changes the user will see?

Currently, if the player agrees to switch a Pokemon at the start of a wave, but then cancels the switch in the party menu, the original Pokemon's post-summon abilities (e.g. Drought/Drizzle) do not apply. This fixes that.

Why am I making these changes?

fixes #3482

What are the changes from a developer perspective?

  • Moved a call to remove a Pokemon's PostSummonPhase from the phase queue from the end of CheckSwitchPhase (i.e. when the player accepts the prompt to switch) to the end of SwitchPhase (i.e. after the player confirms the Pokemon to switch in from the party UI)
  • Fixed some potential sequencing issues in PostSummonPhase that may cause Commander to activate after calling PostSummonPhase.end().

Screenshots/Videos

PokeRogue.-.Google.Chrome.2024-11-18.00-35-29.mp4

How to test the changes?

  1. Start a run with a Pokemon with a post-summon ability as your first starter choice and at least 1 other Pokemon.
  2. When prompted to switch, accept the initial prompt, then back out of the party menu.
  3. Your Pokemon's post-summon effects should trigger after declining the switch.

Checklist

  • I'm using beta as my base branch
  • There is no overlap with another PR?
  • The PR is self-contained and cannot be split into smaller PRs?
  • Have I provided a clear explanation of the changes?
  • Have I considered writing automated tests for the issue?
  • If I have text, did I make it translatable and add a key in the English locale file(s)?
  • Have I tested the changes (manually)?
    • Are all unit tests still passing? (npm run test)
  • Are the changes visual?
    • Have I provided screenshots/videos of the changes?

@innerthunder innerthunder requested a review from a team as a code owner November 18, 2024 09:02
@innerthunder innerthunder added P2 Bug Minor. Non crashing Incorrect move/ability/interaction Ability Affects an ability labels Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ability Affects an ability P2 Bug Minor. Non crashing Incorrect move/ability/interaction
Projects
None yet
3 participants