-
Notifications
You must be signed in to change notification settings - Fork 2
added Version 1.21.4 - 1.21.8 #5
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
base: master
Are you sure you want to change the base?
Conversation
|
Thanks for the pull request! I’ll look into it in a few moments, but with a cursory view it looks like you removed an override without any replacement. Can you explain your reasoning? |
|
Oh, sorry, my bad, i opened this too early, some commits are missing |
|
I've just done a basic test on release 2.0.0 with Minecraft 1.21.8, and it seems to work fine. Are you encountering any issues to warrant the upgrade? |
|
On my PC the Snippet @Overwrite
@Nullable
private Path getDataPackTempDir() {
this.dataPackTempDir = DatapackInstallerClient.MAIN_PATH;
return this.dataPackTempDir;
}when removing the Later in playing it will use the DP folder from the mod OHThis is a lot more complicated then thought before, the new function for I am working on it at the moment |
|
So, I think, I fixed it, the version works and its using the Custom Folder |
|
Weird. I can't seem to reproduce your issue with 1.21.4, either. Which jar from the release are you using? |
|
I used the source code and compiled it myself in Intellij Idea, this is probably the reason. I dont think this would make a difference but i am using Temurium 21 JDK |
|
By "compiled it myself in Intellij Idea," do you mean that you compiled the mod and then used the resulting artifact in game, or you ran the game from your IDE directly? |
...