-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add 1.19.4 Support #528
base: arch-1.19
Are you sure you want to change the base?
Add 1.19.4 Support #528
Conversation
* Architectury updated to 8.1.80 * Architectury Loom updated to 1.2-SNAPSHOT * Gradle updated to 8.1.1 * Shadow updated to 8.1.1 * Omega-Config updated to 1.3.0+1.19.2 * Targeting Fabric loader 0.14.19 * Targeting Fabric API version 0.81.1+1.19.4 * Targeting Forge version 45.0.64 * Requiring Java 17+
Immediate crash when opening the identity menu on
|
This was needed because Forge didn't have method_30255 while Identity did, but now both of them have it so it is no longer necessary.
@0xicl33n I pushed a change to my branch to fix the issue with Forge. You can give it a try again if you want. https://github.com/x64architecture/identity/releases/tag/1.19.4-Dev-Build-2 |
heck yeah ill try it and see if your unoffical update works or has bugs |
thank you this is my favorite mod |
we got 1.19.3 now |
took me a while to get back to this. yeah it works great! |
Hi,
I added support for 1.19.4. I tested it under Fabric and Forge and so far and everything seems okay to me but I haven't used prior versions so I don't really have anything to base it off of.
I am new to working with Minecraft mods so some of the dependency updates may have been unnecessary or may make using lower versions (i.e. 1.19.3) impossible. Let me know if anything needs to be changed.
Potential Problems:
There was a bunch of Frog animation state stuff that was removed in 1.19.3+, I commented out the code but I am not sure what it was supposed to do. When using the frog the limbs properly move and when in water the frog properly does the swim animation, so perhaps they unified it to be like other entities and use generic classes.
The
airStrafingSpeed
property was removed from the RavagerEntity class, I could not figure out what the replacement is, or if there is one.The tooltip stuff could have issues, though they seem to work, again not sure how they worked in prior versions.
If anyone wants to try this out for themselves I have binaries posted here:
(2023-05-24) Updated to fix Forge version:
https://github.com/x64architecture/identity/releases/tag/1.19.4-Dev-Build-2