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

fix error on client when opening multiblock UI for the first time #1042

Merged
merged 2 commits into from
Mar 31, 2024

Conversation

screret
Copy link
Member

@screret screret commented Mar 31, 2024

What

title

Implementation Details

added some null checks, removed useless field, made text only be initialized on the server

Outcome

fixes a logged packet error when opening the UI for the first time

@screret screret requested a review from a team as a code owner March 31, 2024 12:44
@screret screret merged commit 8fd369e into 1.20.1 Mar 31, 2024
1 check passed
@screret screret deleted the sc/fix-multiblock-client-error branch March 31, 2024 15:53
screret added a commit that referenced this pull request Mar 31, 2024
* Update lang file (#1035)

* Translated using Weblate (Russian)

Currently translated at 100.0% (5062 of 5062 strings)

Translation: Gregicality/GregTech Modern
Translate-URL: http://weblate.pantsu.moe/projects/gregicality/gregtech-modern/ru/

* Translated using Weblate (Russian)

Currently translated at 100.0% (5062 of 5062 strings)

Translation: Gregicality/GregTech Modern
Translate-URL: http://weblate.pantsu.moe/projects/gregicality/gregtech-modern/ru/

* Translated using Weblate (Russian)

Currently translated at 99.9% (5063 of 5067 strings)

Translation: Gregicality/GregTech Modern
Translate-URL: http://weblate.pantsu.moe/projects/gregicality/gregtech-modern/ru/

* Translated using Weblate (Russian)

Currently translated at 100.0% (5067 of 5067 strings)

Translation: Gregicality/GregTech Modern
Translate-URL: http://weblate.pantsu.moe/projects/gregicality/gregtech-modern/ru/

* Translated using Weblate (Russian)

Currently translated at 100.0% (5080 of 5080 strings)

Translation: Gregicality/GregTech Modern
Translate-URL: http://weblate.pantsu.moe/projects/gregicality/gregtech-modern/ru/

---------

Co-authored-by: Western01 <[email protected]>

* update LDlib to fix server crashing (#1040)

* fix large assembler and circuit assembler energy hatch counts (#1041)

* fix error on client when opening multiblock UI for the first time (#1042)

---------

Co-authored-by: marisathewitch <[email protected]>
Co-authored-by: Western01 <[email protected]>
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.

2 participants