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

issues#176 fix and minor improvements #786

Closed
wants to merge 4 commits into from

Conversation

modox94
Copy link

@modox94 modox94 commented Aug 23, 2022

What kind of change does this PR introduce?

#176

What is the current behavior?
Tests with errors. Locale cant be changed in ESM project. Deprecated addon in storybook.

What is the new behavior?
Tests ok. Locale ok. Replaced addon for actual.

Checklist:

  • Documentation
  • Tests
  • Ready to be merged

If there are any questions you can text me any time here https://t.me/crazynike94

@arthurdenner
Copy link
Owner

Hi! I appreciate the interest but the proposed changes have the same issue pointed on previous attempts:
It would cause all locales to be bundled with the library.
Apologies for not having the time to look into this library but I'd rather not push this change forward now.

@modox94
Copy link
Author

modox94 commented Sep 9, 2022

Hi, thanks for the answer. I have rewrote logic of localization import, maybe branch is ready for merging now. This branch have breaking changes now, so users to have change code after updating package.

@modox94 modox94 closed this Jun 29, 2023
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.

None yet

2 participants