Skip to content
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

fix: status message typo & outdated error #19

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

chadoh
Copy link
Contributor

@chadoh chadoh commented Apr 1, 2021

  • typo: transfering -> transferring
  • outdated error: referencing "Lock" step instead of "Transfer" step

The outdated error message made me realize that the error messages were not included in the i18n, so I added them there and added a helper to client/i18nHelpers to access it.

* typo: transfering -> transferring
* outdated error: referencing "Lock" step instead of "Transfer" step

The outdated error message made me realize that the error messages were
not included in the i18n, so I added them there and added a helper to
`client/i18nHelpers` to access it.

// get available & fallback locale from given i18n
const availableLocales = Object.keys(i18n)
const fallback = availableLocales[0] as string
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should probably fall back to something that matches the first part of locale here. So if no es_MX but there is an es_ES, use es_ES.

@chadoh chadoh requested a review from paouvrard as a code owner April 21, 2021 07:43
@paouvrard paouvrard marked this pull request as draft May 16, 2021 06: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.

1 participant