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

1.20.6 merge #90

Merged
merged 17 commits into from
Jun 3, 2024
Merged

1.20.6 merge #90

merged 17 commits into from
Jun 3, 2024

Conversation

desht
Copy link
Contributor

@desht desht commented Jun 3, 2024

1.20.6 port, for beta release

desht added 15 commits April 30, 2024 15:47
Compiles and runs, not heavily tested yet

ItemStack "NBT" editing still works, even with components; component data is serialized
to NBT for the NBT editor, and deserialized back to the stack if changes are made

Arch FluidStack.empty() currently broken (returning null, causes crashes in fluid selection
screens)

Won't work with NeoForge 20.6.3-beta or later due to arch/neoforge mappings issues
(see architectury/architectury-loom#207)

No MinecraftForge support at all yet
Older arch network API is deprecated now
Fixes null empty fluid stacks and newest arch loom allows building
with up-to-date NeoForge
Like NeoForgeExtraCodecs method, but we need it cross-platform...
20.6.50+ has API break causing architectury problems
items within a config group are now sorted alphabetically
The textbox now allows text which is considered invalid by the isValid()
method, but will display it in red. Users of the textbox should check its
validity with isTextValid() when deciding if the value should be committed.
Shift-click still adjusts selection pos, as expected

FTBTeam/FTB-Mods-Issues#1185
also added javadocs for all the ConfigGroup add methods
Better support for any future menu-based screens
@desht desht merged commit 6bce739 into main Jun 3, 2024
2 checks passed
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