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

[Hotfix] Allow Fake Out to be used at the start of every wave again #4105

Closed
wants to merge 1 commit into from

Conversation

DayKev
Copy link
Collaborator

@DayKev DayKev commented Sep 8, 2024

What are the changes the user will see?

Fake Out and First Impression can be used at the start of every wave.

Why am I making these changes?

Request from @damocleas due to player feedback.

What are the changes from a developer perspective?

The active Pokémon's turn count is set to 1 at the start of every wave, so first-turn-only moves won't fail.

How to test the changes?

npm run test fake_out

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?

@DayKev DayKev added the Move Affects a move label Sep 8, 2024
@DayKev DayKev requested a review from a team as a code owner September 8, 2024 08:07
Copy link
Collaborator

@innerthunder innerthunder left a comment

Choose a reason for hiding this comment

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

lgtm 🚀

@damocleas
Copy link
Collaborator

Please test with mat block too

@DayKev
Copy link
Collaborator Author

DayKev commented Sep 8, 2024

Mat Block also works.

@DayKev DayKev added the Lead Dev Review Lead Dev Review is required label Sep 9, 2024
@@ -23,6 +23,16 @@ export class BattleEndPhase extends BattlePhase {
this.scene.unshiftPhase(new GameOverPhase(this.scene, true));
}

/**
* Allow Fake Out and First Impression to be used at the start of every wild battle
Copy link
Collaborator

Choose a reason for hiding this comment

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

If you just want to make a change to two specific moves wouldn't this make more sense as a move attribute rather than doing something that might cause unintended behavior by manually changing a Pokemon's turns out?

@DayKev
Copy link
Collaborator Author

DayKev commented Sep 9, 2024

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Lead Dev Review Lead Dev Review is required Move Affects a move
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants