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

feat: add has_potion_effect and potion_effect_level (closes #66) #84

Closed
wants to merge 4 commits into from

Conversation

iGabyTM
Copy link
Member

@iGabyTM iGabyTM commented Jun 18, 2023

Has to be merged after #83

@iGabyTM iGabyTM changed the title feat: add potion_has and potion_level (close #66) feat: add potion_has and potion_level (closes #66) Jun 18, 2023
@iGabyTM iGabyTM linked an issue Jun 18, 2023 that may be closed by this pull request
iGabyTM added 2 commits June 21, 2023 20:49
As @BlitzOffline pointed out, `has_potion_effect` makes more sense than `potion_has`
Because these placeholders requires an argument they need to be outside the switch block
@iGabyTM iGabyTM requested a review from BlitzOffline June 21, 2023 17:51
@iGabyTM iGabyTM changed the title feat: add potion_has and potion_level (closes #66) feat: add has_potion_effect and potion_level (closes #66) Jun 21, 2023
Last edit, I promise 😬
@iGabyTM iGabyTM changed the title feat: add has_potion_effect and potion_level (closes #66) feat: add has_potion_effect and potion_effect_level (closes #66) Jun 21, 2023
@cj89898
Copy link
Contributor

cj89898 commented Jul 1, 2023

Just saw this pop up in the discord, and was just curious about adding has_potion_effect when we already have has_potioneffect.

I noticed extra code was also put in for the new one, instead of them running the same thing code.

@iGabyTM
Copy link
Member Author

iGabyTM commented Jul 1, 2023

Because I did not see it welp

@Tanguygab Tanguygab mentioned this pull request Aug 27, 2023
@iGabyTM
Copy link
Member Author

iGabyTM commented Jun 24, 2024

I've included this (potion_effect_level) placeholder in https://github.com/PlaceholderAPI/Player-Expansion/tree/feature/rewrite

@iGabyTM iGabyTM closed this Jun 24, 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.

SUGGESTION - 2 new placeholders
4 participants