Skip to content

feat: add multilingual support #238

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

Merged
merged 4 commits into from
Jun 8, 2020
Merged

Conversation

marierigal
Copy link
Contributor

@marierigal marierigal commented May 13, 2020

Feature

  • Multilingual support - en | fr

Linked to #128

This is my first contribution to a public project. I need this feature for a work project so I figured this was my chance to get started. I'm not sure I made the right technical choices and I'm open to any suggestions or improvements.

@netlify
Copy link

netlify bot commented May 13, 2020

Deploy Success!

Built with commit 2d8acda

https://deploy-preview-238--identity.netlify.app

@marierigal marierigal force-pushed the feat/multilang branch 3 times, most recently from 79938ac to 27645b7 Compare May 16, 2020 13:53
@shortdiv shortdiv requested a review from erezrokah June 5, 2020 15:36
@erezrokah erezrokah added the type: feature code contributing to the implementation of a feature and/or user facing functionality label Jun 7, 2020
Copy link
Contributor

@erezrokah erezrokah left a comment

Choose a reason for hiding this comment

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

Thanks @marierigal and sorry for taking so long to review this.
I rebased the branch, added some tests, updated the readme to include the optional locale in the init method. added a missing translation key (continue_with) under providers and an example usage to the demo site.

Also, any missing key should first default to the English locale, then to the key.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature code contributing to the implementation of a feature and/or user facing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants