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

Add tests for Defog and fix Defog battle strings #2737

Merged
merged 4 commits into from
Mar 6, 2023

Conversation

DizzyEggg
Copy link
Collaborator

Fixes #1408

Adds tests for Defog.
Fixes incorrect battle strings when hazards are removed after using Defog.
Creates BattleScript_TerrainEnds and TryEndTerrain to avoid duplicating scripts/code.
Adds config and mechanics for Defog to remove terrains.

@Jaizu
Copy link

Jaizu commented Mar 6, 2023

Please add a line jump before BATTLESTRINGS_COUNT

@DizzyEggg
Copy link
Collaborator Author

Done

@SBird1337 SBird1337 merged commit a04046d into rh-hideout:upcoming Mar 6, 2023
BattleScriptExecute(BattleScript_GrassyTerrainHeals);
effect++;
}
effect = TryEndTerrain(STATUS_FIELD_GRASSY_TERRAIN, B_MSG_TERRAINENDS_GRASS);
Copy link
Collaborator

Choose a reason for hiding this comment

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

This removes the call to BattleScript_GrassyTerrainHeals, so it looks to break grassy terrain functionality

@AsparagusEduardo AsparagusEduardo mentioned this pull request May 31, 2023
@DizzyEggg DizzyEggg deleted the defog branch June 23, 2024 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Defog's effect and usage string are on the opponent's mon
4 participants