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

Update Stinky.js #371

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Update Stinky.js #371

wants to merge 4 commits into from

Conversation

coopw1
Copy link
Contributor

@coopw1 coopw1 commented Sep 25, 2023

Clarifying stinky's description, before I really couldn't tell that it also removed current buffs without checking. I think it's pretty important to clarify, as that makes a huge difference in comparing it to other weapons like pstaff.

If anyone thinks they have a better way of wording it, feel free to suggest.

Clarifying stinky's description, before I really couldn't tell that it also removed current buffs without checking. I think it's pretty important to clarify, as that makes a huge difference in comparing it to other weapons like pstaff.
Whoops, I was wrong about it removing - it would be nice to word this more specifically tho
@coopw1
Copy link
Contributor Author

coopw1 commented Sep 25, 2023

Whoops, I was wrong about it removing - it would be nice to word this more specifically tho

@kyljy
Copy link

kyljy commented Sep 25, 2023

I would personally reword to be more like:
Prevents any buffs *AND* debuffs from affecting this animal, and inflicts **?%** ${WeaponInterface.magEmoji}MAG when an effect is successfully blocked.

But if not, please fix the successly/successfully typo :)

@coopw1
Copy link
Contributor Author

coopw1 commented Sep 25, 2023

oops, typing is hard

@coopw1
Copy link
Contributor Author

coopw1 commented Sep 25, 2023

I don't really have a preference either way for the phrasing, i'll ask a few people to see what others think

changed wording
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.

2 participants