Skip to content

Turn groupColor field value into optional field codec#4593

Merged
YoungOnionMC merged 1 commit into1.20.1from
jj/optional-color-codec
Feb 10, 2026
Merged

Turn groupColor field value into optional field codec#4593
YoungOnionMC merged 1 commit into1.20.1from
jj/optional-color-codec

Conversation

@jurrejelle
Copy link
Contributor

What

In #4273, the codec for GTRecipe was changed to include a recipeColor field. This field should have been made optional with a default value of -1, for compatability reasons.

How Was This Tested

{"type":"gtceu:alloy_smelter","duration":100,"category":"gtceu:alloy_smelter","tickInputs":{"eu":[{"content":4}]},"inputs":{"item":[{"content":{"type":"gtceu:sized","count":5,"ingredient":{"item":"minecraft:acacia_planks"}}},{"content":{"type":"gtceu:circuit","configuration":15},"chance":0}]},"outputs":{"item":[{"content":{"type":"gtceu:sized","count":1,"ingredient":{"item":"minecraft:acacia_boat"}}}]}}
image

Additional Information

It is worth noting that support like this for internal structures is an exception, not a rule.

@jurrejelle jurrejelle requested a review from a team as a code owner February 9, 2026 10:59
@jurrejelle jurrejelle added type: bugfix General bug fixes 1.20.1 Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. labels Feb 9, 2026
@github-actions github-actions bot added the Tests: Passed Game Tests have passed on this PR label Feb 9, 2026
@YoungOnionMC YoungOnionMC merged commit f77dbb8 into 1.20.1 Feb 10, 2026
5 of 6 checks passed
@YoungOnionMC YoungOnionMC deleted the jj/optional-color-codec branch February 10, 2026 09:17
Mqrius added a commit to Mqrius/GregTech-Modern that referenced this pull request Feb 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.20.1 Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. Tests: Passed Game Tests have passed on this PR type: bugfix General bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants