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

Fix logs for protosynthesis and quarkdrive #10300

Closed
wants to merge 4 commits into from

Conversation

Jerick120
Copy link
Contributor

@Jerick120 Jerick120 commented May 15, 2024

Fix incorrect logs with stat name appended to the effect (for example protosynthesisdef) requiring an explicit if-else check and string slicing for protosynthesis and quarkdrive on the client side.
This needs my PR smogon/pokemon-showdown-client#2248 on the client side to be merged at the same time in order for everything to work.

Comment on lines 4610 to 4615
let activated = false;
for (const target of pokemon.adjacentFoes()) {
if (!activated) {
this.add('-ability', pokemon, 'Supersweet Syrup', 'boost');
activated = true;
}
Copy link
Member

Choose a reason for hiding this comment

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

This does not appear to be a documented change.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was already merged here: #10296 (comment)

Copy link
Member

Choose a reason for hiding this comment

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

Sounds like it needs to be rebased/merged, then? I can do that for you if that's difficult.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh sure thing! thankyou!

@Jerick120 Jerick120 closed this by deleting the head repository May 17, 2024
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.

None yet

2 participants