Closed kem226 closed 6 years ago
Ready for land after conflicts resolved
whoops, we have to adapt the tests to work with internationalization. Will plan to work on that today unless you guys know of a quick-fix.
If not, since I don't want that to be a blocker for this PR, might I suggest we remove the internationalization component from the RequestInviteForm
component for now? We can add back in once tests have been adapted.
followed @markulrich's link and added intl-enzyme-test-helper
to the repo. we're not currently testing i18n translations, but this sets us up to be able to provide messages
to an IntlProvider
and ensure it works.
Awesome, thank @ataki !!
Hit close instead of merge ... facepalm
Added use of the Library https://github.com/yahoo/react-intl for internationalization support. Currently only english support but allows us to store strings in a separate file that can also be translated later as we want to support more languages.
issue #26