-
Notifications
You must be signed in to change notification settings - Fork 475
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
feat: locale message file cache configuration #2357
Merged
kazupon
merged 7 commits into
nuxt-modules:next
from
BobbieGoede:feat/locale-message-file-cache-configuration
Sep 4, 2023
Merged
feat: locale message file cache configuration #2357
kazupon
merged 7 commits into
nuxt-modules:next
from
BobbieGoede:feat/locale-message-file-cache-configuration
Sep 4, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kazupon
reviewed
Sep 2, 2023
BobbieGoede
force-pushed
the
feat/locale-message-file-cache-configuration
branch
from
September 2, 2023 15:26
21c0a80
to
32c5691
Compare
β Live Preview ready!
|
BobbieGoede
force-pushed
the
feat/locale-message-file-cache-configuration
branch
from
September 4, 2023 09:25
a3b30a0
to
7999daa
Compare
kazupon
approved these changes
Sep 4, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great works!
Thank you so much!
DarthGigi
pushed a commit
to DarthGigi/i18n
that referenced
this pull request
Apr 16, 2024
* feat: locale message file cache configuration * test: fix types and failing errors * fix: nuxtInternalOptions `__normalizedLocales` * refactor: simplify types * refactor: remove duplicate utility function * docs: add caching section to lazy-load translations guide * test: add test to assert cache configuration is used
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
π Linked issue
#2185
#2293
β Type of change
π Description
Resolves #2293
Changes locale message file configuration,
file
andfiles
on locales can now be set to{ path: string; cache?: boolean }
, if cache is left unset caching be enabled by default for locale files that are notdynamic
.It was a bit difficult to think of a proper way to test whether cache configuration is taking effect as it happens internally, the added test checks this based on debug logs, perhaps there is a better way to do this.
π Checklist