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

Update to 1.20.4 #500

Open
wants to merge 6 commits into
base: multiloader/1.20.X
Choose a base branch
from

Conversation

JulianVennen
Copy link

This PR updates Controllable to Minecraft 1.20.4

Both Fabric and Forge have been tested and work correctly.
Please note that the ForgeGradle runs appear to be broken, causing a crash in the development environment.
Compiling a jar and running it in a normal Minecraft instance is unaffected.

If supporting NeoForge as well is desired, please let me know and I will implement that too.

@JulianVennen JulianVennen marked this pull request as draft March 4, 2024 17:47
@JulianVennen
Copy link
Author

It appears that there are still some issues with FabricClientHelper/ForgeClientHelper, I'll probably take a closer look at those tomorrow

@MaxedOut655
Copy link

MaxedOut655 commented Mar 4, 2024

I'd just like to let you know about a crash this causes when opening the bindings in the menu, here's the crash report:

The game crashed whilst rendering screen
Error: java.lang.ExceptionInInitializerError

The full crash report is here: https://pastebin.com/JqHv936F

Thanks for all your work updating it!

@JulianVennen JulianVennen marked this pull request as ready for review March 5, 2024 12:12
@JulianVennen
Copy link
Author

This crash (and various similar ones) should be fixed now. I replaced the usages of reflection that were causing this with access transformer entries, so this will be noticeable during compile time if it breaks again in a future update.

@MaxedOut655
Copy link

The menu to rebind keys on fabric at least is very dark, and it doesn't even allow you the option to add external bindings but it lets you see them, unlike before where you could add them.

@JulianVennen
Copy link
Author

JulianVennen commented Mar 8, 2024

The dark menu should be fixed now, it appears that the background gradient was already being drawn by the parent class.

@MaxedOut655
Copy link

MaxedOut655 commented Mar 8, 2024

There is just that one final thing, adding external mods' bindings.

Here is what it currently looks like, there is supposed to be a plus to add in the bindings indvidually. The menu gradients look perfect now though! Thanks again for the hard work.

image

@JulianVennen
Copy link
Author

I'm not sure how that broke, but I also didn't take a closer look at that yet.

Here are the jar files for anyone who wants to use the mod in the current state: Fabric Forge

@terickson001
Copy link

I'm interested in adding NeoForge support to this PR, but I cannot get the maven repository for Framework to behave. Is there some important setup I'm missing?

@SilentException
Copy link

I'm interested in adding NeoForge support to this PR, but I cannot get the maven repository for Framework to behave. Is there some important setup I'm missing?

You need to add personal access token with read:packages scope and then set GPR_USER=your@email and GPR_KEY=token environment variables or gpr.user=your@email and gpr.key=token in gradle.properties...

@terickson001
Copy link

Ah my mistake, I was using my github username not e-mail, in combination with, I assume, regenerating the token but not saving it. Thanks

@terickson001
Copy link

@JulianVennen How did you solve the issue with AbstractContainerScreen.slotClicked being changed from protected to public through the Access Transformers? I'm getting errors about it's subclasses having weaker access privileges.

@JulianVennen
Copy link
Author

You need to reload the Gradle project in your IDE every time you modify the access transformer.

You probably also need to tell neogradle the path to your access transformer if you are talking about your neoforge project.

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.

4 participants