Skip to content

Add world conversion warning screen#210

Open
SuperSoupr wants to merge 7 commits into
masterfrom
wc-gui
Open

Add world conversion warning screen#210
SuperSoupr wants to merge 7 commits into
masterfrom
wc-gui

Conversation

@SuperSoupr

Copy link
Copy Markdown
Contributor

Depends on GTNewHorizons/GTNHLib#409, need to update deps when it's merged

image

# Conflicts:
#	dependencies.gradle
#	src/main/java/com/fouristhenumber/utilitiesinexcess/UtilitiesInExcess.java
#	src/main/resources/assets/utilitiesinexcess/lang/en_US.lang
@SuperSoupr

Copy link
Copy Markdown
Contributor Author

Activation logic for this needs to be discussed, not sure what was decided in the end
Are we having a warning all the time or just when not in nh?

@FourIsTheNumber

Copy link
Copy Markdown
Collaborator

Activation logic for this needs to be discussed, not sure what was decided in the end Are we having a warning all the time or just when not in nh?

Probably best to ask admins or poll in dev votes

@boubou19

boubou19 commented Jul 5, 2026

Copy link
Copy Markdown
Member

If we look into what NH does already, you'll see that we only expose a warning when there are unhandled missing mapping events fired. FPLib added another message when the id extenders have changed, copying the fml missing mapping screen. But if your goal is to do a seamless transition on NH, the screen should not be visible imo.

Also, no matter what you end up chosing, i suggest to make it so that screen behaves the same as regular forge screen, so server admins can control it with -Dfml.doNotBackup and -Dfml.queryResult flags.

@FourIsTheNumber

Copy link
Copy Markdown
Collaborator

I think disabling it in nh is likely fine, but we can config it on by default.

@SuperSoupr

SuperSoupr commented Jul 5, 2026

Copy link
Copy Markdown
Contributor Author

Yeah, I agree with disabling it in nh but keeping the default on

i suggest to make it so that screen behaves the same as regular forge screen, so server admins can control it with -Dfml.doNotBackup and -Dfml.queryResult flags.

It piggybacks off of the fml query system so those flags will already work for it too :)

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.

3 participants