Skip to content

Conversation

sawyerh
Copy link
Contributor

@sawyerh sawyerh commented Oct 27, 2022

Changes

  • Remove the resources option from Storybook's i18next initialization. We're utilizing the i18next-http-backend package for loading locales in Storybook via HTTP requests, so manually setting them doesn't appear to be required for it to render the strings.
  • Update Jest's i18next initialization to dynamically set resources based on the file structure. This removes the need for updating the Jest setup file every time a new language file is created.
  • Fix the test-update script

@sawyerh sawyerh requested review from jimmyfagan and aligg October 27, 2022 23:33
Copy link
Contributor

@jimmyfagan jimmyfagan left a comment

Choose a reason for hiding this comment

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

Nice, looks good to me!

@sawyerh sawyerh merged commit 6599ff6 into main Oct 28, 2022
@sawyerh sawyerh deleted the sawyerh/dynamic-i18n-loading branch October 28, 2022 01:03
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