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 for Dancer activating when enemy in not on field / using a 2 steps charging move #1708

Merged
merged 4 commits into from
Jun 5, 2024

Conversation

YounesM
Copy link
Contributor

@YounesM YounesM commented Jun 1, 2024

What are the changes?

Fix for #1686 and #1450

What did change?

Fixed a bug where Dancer activates while a Pokemon with Dancer Ability is charging a 2 turns move.
Fixed a bug where Dancer activates while a Pokemon with Dancer Ability is on the opponent team but not on the field.

Screenshots/Videos

Before

2024-06-02.01-08-21.mp4

After

2024-06-02.01-09-14.mp4

How to test the changes?

Have a pokemon having the Dancer abilty use fly and use a dancing move afterwards.
Have a pokemon having the Dancer abilty on the opponent team but not in the field and use a dancing move.

Checklist

  • 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 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?

@Tempo-anon Tempo-anon added (Legacy) Bug Legacy Label, don't apply to new issues/PRs Ability Affects an ability labels Jun 2, 2024
Copy link
Collaborator

@Tempo-anon Tempo-anon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there's still some edge cases with dancer but this PR should at least fix the case of fly/dig/dive/shadowforce which are pretty common

@Tempo-anon Tempo-anon merged commit e614aec into pagefaultgames:main Jun 5, 2024
3 checks passed
Jason954 pushed a commit to Jason954/pokerogue that referenced this pull request Jun 6, 2024
…2 steps charging move (pagefaultgames#1708)

* Fixes !1686 and !1450

* Added forbidden tags

* Restored original import indentations

* Restored missing import
Korwai pushed a commit to Korwai/pokerogue that referenced this pull request Jun 14, 2024
…2 steps charging move (pagefaultgames#1708)

* Fixes !1686 and !1450

* Added forbidden tags

* Restored original import indentations

* Restored missing import
SavGRY pushed a commit to SavGRY/pokerogue that referenced this pull request Jun 16, 2024
…2 steps charging move (pagefaultgames#1708)

* Fixes !1686 and !1450

* Added forbidden tags

* Restored original import indentations

* Restored missing import
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ability Affects an ability (Legacy) Bug Legacy Label, don't apply to new issues/PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants