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

Metallurgy 1.21: Lore Line Shrink #1078

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

EpyonProjects
Copy link
Member

Currently, Metallurgy shamir lore takes up 3 lines

Name: Obsidian Cast
Lore: Band
Shamir

This update removes that standalone "Shamir" line to only use 2 lines
Name: Obsidian Cast
Lore: Band
Shamir

This change is part of the larger Metallurgy redesign.
Existing items will not have " Shamir" appended to the end, but will function the same. This also means old unattached bands will not be stackable with shamirs created after this update.

@misode misode added feature-update An additional feature has been added to a module needs-testing Requires in-game testing labels Dec 1, 2024
@@ -2,12 +2,30 @@
# run from smooshing/remove_band/check

execute if score custom_model_data gm4_ml_data matches 0 run data remove entity @s Item.components."minecraft:custom_model_data"
execute if items entity @s contents *[custom_data~{gm4_metallurgy:{active_shamir:"infinitas"}}] run data remove entity @s Item.components."minecraft:max_stack_size"
Copy link
Member

Choose a reason for hiding this comment

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

Why does infinitas need special handling? Maybe include a comment about this in the function

@Bloo-dev Bloo-dev marked this pull request as draft January 19, 2025 20:09
@Bloo-dev
Copy link
Member

This is now waiting on some 1.21.5 changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-update An additional feature has been added to a module needs-testing Requires in-game testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants