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

Export message merging utilities or move to shared #1634

Closed
3 tasks done
BobbieGoede opened this issue Nov 14, 2023 · 1 comment · Fixed by #1635
Closed
3 tasks done

Export message merging utilities or move to shared #1634

BobbieGoede opened this issue Nov 14, 2023 · 1 comment · Fixed by #1635
Labels
Status: Proposal Request for comments

Comments

@BobbieGoede
Copy link
Member

Clear and concise description of the problem

We redefine/reimplement message merging in @nuxtjs/i18n, to prevent bugs and keep things stable it would be beneficial to make the message merging available for packages downstream.

Suggested solution

Either export the relevant utility functions from vue-i18n or maybe these can be moved to @intlify/shared instead?

Alternative

No response

Additional context

I'm pretty sure there is a bug in the message merging in @nuxtjs/i18n

Validations

@BobbieGoede BobbieGoede added the Status: Proposal Request for comments label Nov 14, 2023
@kazupon
Copy link
Member

kazupon commented Nov 15, 2023

@BobbieGoede

Either export the relevant utility functions from vue-i18n or maybe these can be moved to @intlify/shared instead?

I think it should be moved to @intlify/shared
You can push the PR :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Proposal Request for comments
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants