-
Notifications
You must be signed in to change notification settings - Fork 17
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.1 #90
base: 1.19.x
Are you sure you want to change the base?
Update to 1.20.1 #90
Conversation
Update en_us.json as changes do not get applied currently when saved: Stacker.java line 32 "stackerConfig.registerSaveListener((configHolder, stackerConfig1) ->{ loadStacker("save");"
Andrew (mod author) at least for now is not interested in update/maintain stacker, so the only solution is that we as users of stacker try to maintain it updated, the mod saves the config, its not necessary close the entirely game, only close the world and reopen it, in my last commentary i uploaded the stacker version with stacc api updated, it works in 1.20.4, i have been playing for a week without problems with it, so try to use. |
Can someone give me either a tutorial on how to build this so I can have the .jar or upload the built .jar somewhere? |
I have made my own personal use build available on my fork of this mod. NOTE: You can find it here |
Thank you for releasing this. I've been meaning to learn how to develop mods but work and life don't leave so much free time. I really appreciate it. |
I tried it with stacc, fabric api and cloth config only but didn't work. |
@ppyLEK |
Update to Minecraft 1.20.1
Update gradle.
Mod seems to work fine other than when saving config, changes only apply after /reload or restart.
This does get called but it seems either Minecraft or Fabric changed something breaking it.
stackerConfig.registerSaveListener((configHolder, stackerConfig1) -> { loadStacker("save");
As settings are mostly "set it and forget it" type of deal I have changed
en_us.json
to indicate that /reload or restart is required.#87