Skip to content

tests: added i18next example #399

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

Conversation

viniciusavieira
Copy link
Contributor

What: Adding i18Next example with locale

Why: #390

How: I've put the logic to display to users how make test with locale in using i18next, I've create an example with a test.
Refs:
https://react.i18next.com/misc/testing
https://codesandbox.io/s/testing-library-i18next-vot1o

Checklist:

  • Documentation added to the
    docs site
  • [ X ] Tests
  • Typescript definitions updated
  • Ready to be merged

Once this PR is accepted I'll updated the docs site.

@viniciusavieira viniciusavieira changed the title tests: updated i18next example tests: added i18next example Jun 28, 2019
Copy link
Member

@kentcdodds kentcdodds left a comment

Choose a reason for hiding this comment

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

I'd actually like to move all the examples from this repo to this one: https://codesandbox.io/s/github/kentcdodds/react-testing-library-examples

That way we can keep the deps in this project slimmed down to only those that are needed for the project.

Could you make this PR to that other repo?

@kentcdodds kentcdodds closed this Jun 28, 2019
@viniciusavieira
Copy link
Contributor Author

viniciusavieira commented Jun 28, 2019

Sure, no problem.

I'm just a bit confused, should I send the PR to this repo, https://github.com/kentcdodds/react-testing-library-examples right?

@kentcdodds
Copy link
Member

Sorry, have the wrong link. That's correct 👍

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