Skip to content

Conversation

ShaneBeee
Copy link
Contributor

Description

This PR aims to fix an issue where type of %itemtype% adds a damage tag to non-damageable items.
(See #7687 for the full story)


Target Minecraft Versions: any
Requirements: none
Related Issues: #7687

@ShaneBeee ShaneBeee changed the base branch from master to dev/patch March 8, 2025 20:08
@Efnilite Efnilite added the bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. label Mar 8, 2025
@sovdeeth
Copy link
Member

sovdeeth commented Mar 8, 2025

I do fear this doesn't solve anything if the source item is damaged, it just solves it for undamaged items

Copy link
Member

@sovdeeth sovdeeth left a comment

Choose a reason for hiding this comment

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

further confirmation that it doesn't affect existing behavior here
https://discord.com/channels/988998880794402856/989227647060041780/1348037377427243093

@Efnilite Efnilite added the patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. label Mar 10, 2025
@sovdeeth sovdeeth requested a review from a team as a code owner March 19, 2025 19:21
@sovdeeth sovdeeth merged commit 2ec7b56 into SkriptLang:dev/patch Mar 19, 2025
5 checks passed
@ShaneBeee ShaneBeee deleted the shane/fix-itemtype-copy branch March 19, 2025 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants