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

All worlds have experimental settings with NeoForge #846

Closed
Lolothepro opened this issue Apr 24, 2024 · 6 comments · Fixed by #901
Closed

All worlds have experimental settings with NeoForge #846

Lolothepro opened this issue Apr 24, 2024 · 6 comments · Fixed by #901
Labels
1.20.5 Targeted at Minecraft 1.20.5 bug A bug or error

Comments

@Lolothepro
Copy link
Contributor

NeoForge Version: 20.5.0-beta

Steps to Reproduce:

  1. Try to generate a new world without any experimental settings
  2. The experimental settings message is displayed.

Description of issue:
All worlds have experimental settings with NeoForge

2024-04-24.16-44-08.mp4
@Lolothepro Lolothepro added the triage Needs triaging and confirmation label Apr 24, 2024
@lukebemish
Copy link
Contributor

lukebemish commented Apr 25, 2024

This will be fixed by adding a KnownPack for mod resource packs, which is doable (though the vanilla code path is a bit funky -- it might need a bit more than this) -- I plan to implement it once #735 is merged, as that PR sets up the infrastructure needed for hidden packs with KnownPacks to behave sensibly.

@sciwhiz12 sciwhiz12 added bug A bug or error 1.20.5 Targeted at Minecraft 1.20.5 and removed triage Needs triaging and confirmation labels Apr 26, 2024
@lukebemish
Copy link
Contributor

Should be fixed -- at least in many cases -- by #901.

@sciwhiz12
Copy link
Member

"In many cases"? What edge cases aren't covered by that PR?

@lukebemish
Copy link
Contributor

lukebemish commented May 1, 2024

  • Mod added dimensions, which are always experimental it seems
  • Stuff added by packs in AddPackFindersEvent that do not add KnownPacks
  • IDK, whatever other cases I've missed -- that lifecycle is collected from like 50 different places

But nothing in neoforge itself, just stuff mods would add.

@sciwhiz12
Copy link
Member

Ah, but the PR will at least fix this issue, which occurs with NeoForge-only.

@sciwhiz12 sciwhiz12 linked a pull request May 1, 2024 that will close this issue
@lukebemish
Copy link
Contributor

lukebemish commented May 1, 2024

Yes, and will fix most mod-caused equivalents -- any caused by the more "normal" non-dynamic datapack registries, from mod built-in datapacks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.20.5 Targeted at Minecraft 1.20.5 bug A bug or error
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants