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

[Unticketed] Update package.json to remove dep i18n #3229

Merged
merged 2 commits into from
Dec 16, 2024
Merged

Conversation

acouch
Copy link
Collaborator

@acouch acouch commented Dec 16, 2024

Summary

Fixes NA

Time to review: 5 mins

Changes proposed

Remove deprecated i18n packages.

Copy link
Collaborator

@doug-s-nava doug-s-nava left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

everything seems to run fine

✅ ✅ ✅ ✅ ✅

I do wonder what plans, if any, we have for translations on storybook though. I assume the idea was that the storybook-react-i18next package was going to be providing something there, and I don't see anything else lined up to take its place...

@acouch
Copy link
Collaborator Author

acouch commented Dec 16, 2024

@doug-s-nava good question. Looks like this might be the way to go with that: https://github.com/stevensacks/storybook-next-intl I created a ticket: #3236

@acouch acouch merged commit 2eee937 into main Dec 16, 2024
13 checks passed
@acouch acouch deleted the acouch/remove-i18n branch December 16, 2024 18:40
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