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

Repairing changes to Limbo options #436

Open
wants to merge 3 commits into
base: 1.20.1-multiplatform
Choose a base branch
from

Conversation

Cephelo
Copy link
Contributor

@Cephelo Cephelo commented Aug 17, 2024

Some changes (presumably untested changes) were made to the new Limbo-related config options I added in my last PR, which caused these options to no longer work, and for the options to reset on game launch like before. I have reverted these changes, and I've verified they work via testing. This should also fix issue #435.
Apologies if I sound rude, that is not my intention; I only want things to work as intended. Thanks!

Repairing changes
Repairing changes
@JustSimplyGaming
Copy link

May i ask when u upload the new Version for tests? :)

@unilock
Copy link

unilock commented Sep 1, 2024

For the record, you can just use

ResourceLocation.tryParse(DimensionalDoors.getConfig().getLimboConfig().escapeTargetWorld

instead of

new ResourceLocation(DimensionalDoors.getConfig().getLimboConfig().escapeTargetWorld.split(":")[0], DimensionalDoors.getConfig().getLimboConfig().escapeTargetWorld.split(":")[1])

@Cephelo
Copy link
Contributor Author

Cephelo commented Sep 22, 2024

Replying to a deleted comment, but yes this should still work - the only two files I've changed in this PR haven't changed since before this PR was made.

I'm also bumping this PR since it's remained stale for just over a month now.

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