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

Make multiblocks able to better modify their NEI preview #3792

Merged
merged 4 commits into from
Jan 13, 2025

Conversation

Lyfts
Copy link
Member

@Lyfts Lyfts commented Jan 12, 2025

Needs (but does not depend on) GTNewHorizons/BlockRenderer6343#26 to work

This aims to get rid of the janky workarounds that have been implemented to change the NEI preview in various multiblocks.
The interface is implemented in GT5U because I didn't feel like hard depping blockrenderer or using @Optional.Interface in every class.

The janky workaround in God forge has been completely removed as it is no longer needed after GTNewHorizons/BlockRenderer6343#25

While trying to fix the piston in EIC I noticed that survival autobuild of those blocks were never implemented, this messed up the structure check when trying to build it and would cause way too many hatches to be placed.
That has been fixed and trying to autobuild an EIC will now result in a working one and preview will also correctly show the piston blocks.
Closes GTNewHorizons/GT-New-Horizons-Modpack#13795

@Lyfts Lyfts requested a review from a team January 12, 2025 14:56
@Lyfts Lyfts added refactor For PRs rewritting a part of the code to have a nicer code overall. bug fix Fix a bug. Please link it in the PR. labels Jan 12, 2025
@serenibyss serenibyss enabled auto-merge (squash) January 13, 2025 07:36
@serenibyss serenibyss merged commit 0c194e2 into master Jan 13, 2025
5 checks passed
@serenibyss serenibyss deleted the nei-preview-modify branch January 13, 2025 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix Fix a bug. Please link it in the PR. refactor For PRs rewritting a part of the code to have a nicer code overall.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EIC structure shown in NEI does not have the center 5 blocks
3 participants