You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, we use LocaleRequired in our component, HOC, and hook to load translated/localized files. The current name can yield a bit of confusion, since you are not loading a locale, but rather a localized message file. Technically, locale is required, which determines which version of the translated file to load, but the purpose of the components is to specify which translated file.
So, instead of withLocaleRequired, should we change these to be withTranslation, or withLocalizedFile, or withLocaleMessages?
Package(s): @gasket/react-intl
AC:
Vet this question with current users and document
Proposal for next steps if any.
The text was updated successfully, but these errors were encountered:
Description
Currently, we use LocaleRequired in our component, HOC, and hook to load translated/localized files. The current name can yield a bit of confusion, since you are not loading a locale, but rather a localized message file. Technically, locale is required, which determines which version of the translated file to load, but the purpose of the components is to specify which translated file.
So, instead of
withLocaleRequired
, should we change these to bewithTranslation
, orwithLocalizedFile
, orwithLocaleMessages
?Package(s): @gasket/react-intl
AC:
The text was updated successfully, but these errors were encountered: