-
Notifications
You must be signed in to change notification settings - Fork 462
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: layer vue i18n config merging #2358
feat: layer vue i18n config merging #2358
Conversation
Iβve checked your PR. |
β Live Preview ready!
|
@kazupon |
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.
Your PR is perfect! π―
Great job!
Thank you so much!
* feat: layer vue i18n config merging * test: update snapshot * fix: layer vuei18n configurations merge order * test: disable jit compilation for vuei18n layer test * docs: describe VueI18n option merging on layers page
π Linked issue
β Type of change
π Description
Based on #2347 (comment) I took a closer look at the current merging of layer VueI18n configurations, it would make sense to merge more of the configuration when possible.
This PR is still a draft, this still needs tests and if necessary will expand the documentation. I made this a bit quick and dirty so it may contain obvious errors π
π Checklist