Skip to content

Remove the need for NBTs on Thermal Augments#2510

Merged
Xefyr0 merged 5 commits intoThePansmith:mainfrom
Blackalf:Monifactory
Mar 14, 2026
Merged

Remove the need for NBTs on Thermal Augments#2510
Xefyr0 merged 5 commits intoThePansmith:mainfrom
Blackalf:Monifactory

Conversation

@Blackalf
Copy link
Copy Markdown
Contributor

@Blackalf Blackalf commented Mar 4, 2026

Modify the Thermal Augments Item Properties on startup instead of adding NBTs to recipes.

Players can now successfully pick up augments from EMI
All previously crafted augments will be automatically updated if their stats are changed (Unless they already have NBT)

Copy link
Copy Markdown
Collaborator

@WithersChat WithersChat left a comment

Choose a reason for hiding this comment

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

I remember you saying you tested this. If this works well then we good.

@Blackalf
Copy link
Copy Markdown
Contributor Author

Blackalf commented Mar 5, 2026

I tested it on my world and could properly get the augments with the right stat buffs using EMI Cheat Mode or by crafting the items. They also properly updated when modifying the augment's stats.

Temporary 1/1 recipes to reset old augment's nbts could be added if wanted

@WithersChat
Copy link
Copy Markdown
Collaborator

Please add those recipes 🥺

@Blackalf
Copy link
Copy Markdown
Contributor Author

All done! :D

Copy link
Copy Markdown
Collaborator

@Xefyr0 Xefyr0 left a comment

Choose a reason for hiding this comment

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

Looks great.
Since this gets rid of the only use of thermal.emi.tooltip would you mind getting rid of that too in each lang file? They're located in kubejs/assets/kubejs/lang/.

@Xefyr0 Xefyr0 merged commit 89fbb8b into ThePansmith:main Mar 14, 2026
1 check passed
@Blackalf Blackalf deleted the Monifactory branch March 14, 2026 16:27
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.

3 participants