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

refactor: lazy loading translation files #4344

Merged
merged 3 commits into from
Nov 6, 2024

Conversation

MukjepScarlet
Copy link
Contributor

No description provided.

@1zun4 1zun4 modified the milestone: 0.17.0 Nov 4, 2024
@1zun4
Copy link
Member

1zun4 commented Nov 4, 2024

This could introduce lag when it's being loaded. Loading all of them is not ideal, but I don't think it makes sense to only load when it's needed. Maybe at least load english since it's common language and the language that is specified.

@MukjepScarlet
Copy link
Contributor Author

en_us and the current language (if present) will be loaded at client starting

@1zun4
Copy link
Member

1zun4 commented Nov 4, 2024

en_us and the current language (if present) will be loaded at client starting

Really? But where? Am I blind?

@MukjepScarlet
Copy link
Contributor Author

In my test, the en_us and current language will be loaded (en_us at first), probably because of the getCommonLanguage call

@1zun4
Copy link
Member

1zun4 commented Nov 4, 2024

Might specifiy it at the ClientResourceReloader

@MukjepScarlet
Copy link
Contributor Author

Recovered

@1zun4 1zun4 added this to the 0.18.0 milestone Nov 5, 2024
@1zun4 1zun4 merged commit 7a7c9b2 into CCBlueX:nextgen Nov 6, 2024
2 checks passed
@MukjepScarlet MukjepScarlet deleted the refactor/lang branch November 6, 2024 12:25
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.

2 participants