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

Added translations as primary dimName source #19

Merged
merged 3 commits into from
Mar 14, 2024

Conversation

AnrDaemon
Copy link
Contributor

No description provided.

@CLAassistant
Copy link

CLAassistant commented Mar 13, 2024

CLA assistant check
All committers have signed the CLA.

@mysticdrew
Copy link
Member

CLA needs to be signed

@mysticdrew
Copy link
Member

Also, can you add a before and after screenshot of what this PR accomplishes?

@AnrDaemon AnrDaemon marked this pull request as draft March 13, 2024 23:01
@AnrDaemon
Copy link
Contributor Author

Also, can you add a before and after screenshot of what this PR accomplishes?

Missing dimension provider data (unchanged from original behavior):
image

Original behavior (extra "overworld/superflat" dimension using FE multiworld module):
image

New behavior (extra dimension name is taken from custom resource pack):
image

@AnrDaemon
Copy link
Contributor Author

I'm pondering an improvement to the caching mechanics.
It should purge the cache on server reconnect and resource pack reload.

@AnrDaemon AnrDaemon marked this pull request as ready for review March 14, 2024 09:09
@mysticdrew
Copy link
Member

I'm pondering an improvement to the caching mechanics. It should purge the cache on server reconnect and resource pack reload.

We do not need a cache honestly.
It does not constantly poll this method.

@AnrDaemon
Copy link
Contributor Author

Judging by the log (Constants.getString() log failures), it does.

@AnrDaemon AnrDaemon requested a review from mysticdrew March 14, 2024 13:35
@mysticdrew
Copy link
Member

Judging by the log (Constants.getString() log failures), it does.

it shouldn't, I will fix that

@mysticdrew
Copy link
Member

nevermind, I cannot easily fix that without some major refactoring. go ahead with your cache

@mysticdrew
Copy link
Member

I'm pondering an improvement to the caching mechanics. It should purge the cache on server reconnect and resource pack reload.

purge it in
JourneymapClient#reset method.

Then somewhere you'll want to add.

        ((IReloadableResourceManager)  Minecraft.getMinecraft().getResourceManager()).registerReloadListener(iResourceManager -> {
           // purge cache
        });

@mysticdrew mysticdrew merged commit b226e66 into TeamJM:master Mar 14, 2024
2 checks passed
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