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.

Sign up for GitHub

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

Jump to bottom

Glow Ink Sac #4239

Closed
1 task done
UsainSrht opened this issue Jul 31, 2021 · 4 comments
Closed
1 task done

Glow Ink Sac #4239

UsainSrht opened this issue Jul 31, 2021 · 4 comments
Labels
completed The issue has been fully resolved and the change will be in the next Skript update. enhancement Feature request, an issue about something that could be improved, or a PR improving something. priority: lowest "Nice to have" updates that are not required (tiny low impact bug fixes or QoL enhancements).

Comments

@UsainSrht
Copy link

Skript/Server Version

Screenshot_2

Bug Description

Skript drops glow ink sac as ink sac with enchant glow but gives glow ink sac as normal glow squid drop.

Expected Behavior

I expect drops normal glow squid drop

Steps to Reproduce

drop glow ink sac at player
give glow ink sac to player

Errors or Screenshots

Screenshot_1

Other

No response

Agreement

  • I have read the guidelines above and confirm I am following them with this report.
@LowScarlet
Copy link

I don't know it yet but it seems reasonable

@LowScarlet
Copy link

you trick it with string parsed it as itemtype

@TheRealUnderscore
Copy link

Skellett has glow item syntax, which seems to be conflicting with Skript's aliases.

@TPGamesNL
Copy link
Member

The glow item expression is not Skript's, so this isn't a Skript issue. The two effects behaving differently is odd, but that will be fixed in #4260, so I'll keep this issue open for that.

@TPGamesNL TPGamesNL added enhancement Feature request, an issue about something that could be improved, or a PR improving something. priority: lowest "Nice to have" updates that are not required (tiny low impact bug fixes or QoL enhancements). PR available Issues which have a yet-to-be merged PR resolving it labels Aug 15, 2021
@TPGamesNL TPGamesNL mentioned this issue Aug 15, 2021
Merged
@TPGamesNL TPGamesNL added completed The issue has been fully resolved and the change will be in the next Skript update. and removed PR available Issues which have a yet-to-be merged PR resolving it labels Mar 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
completed The issue has been fully resolved and the change will be in the next Skript update. enhancement Feature request, an issue about something that could be improved, or a PR improving something. priority: lowest "Nice to have" updates that are not required (tiny low impact bug fixes or QoL enhancements).
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@APickledWalrus @TPGamesNL @TheRealUnderscore @UsainSrht @LowScarlet