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

Missing translations for custom text on crypto wallet #6289

Closed
srirambv opened this issue Oct 2, 2019 · 6 comments · Fixed by brave/ethereum-remote-client#85
Closed

Missing translations for custom text on crypto wallet #6289

srirambv opened this issue Oct 2, 2019 · 6 comments · Fixed by brave/ethereum-remote-client#85

Comments

@srirambv
Copy link
Contributor

srirambv commented Oct 2, 2019

Description

All custom text on crypto wallet is not translated. Found while testing #5686

Steps to Reproduce

  1. Set device locale to French
  2. Create a new profile and launch Crypto wallet
  3. Text coming from MM is translated, custom text implemented by us isn't

Actual result:

Disclosure message
image
Wallet setup selection
image
Password warning details
image

Expected result:

All text should be translated

Reproduces how often:

Easy

Brave version (brave://version info)

Version/Channel Information:

  • Can you reproduce this issue with the current release? NA
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the dev channel? Yes
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? NA
  • Does the issue resolve itself when disabling Brave Rewards? NA
  • Is the issue reproducible on the latest version of Chrome? NA

Miscellaneous Information:

cc; @bbondy @tomlowenthal @kjozwiak

@srirambv
Copy link
Contributor Author

srirambv commented Oct 3, 2019

+1 from @SergeyZhukovsky

@ryanml ryanml self-assigned this Oct 5, 2019
@ryanml
Copy link
Contributor

ryanml commented Oct 5, 2019

The welcome page issues are covered in this pull: brave/ethereum-remote-client#78

As for the disclosure message and modal - these are using translated strings but were added after last rounds of translations are ordered, so next round will fix: cc: @srirambv @bbondy

bbondy added a commit to brave/ethereum-remote-client that referenced this issue Oct 16, 2019
@bbondy bbondy added this to the Ethereum Remote Client 1.0.11 milestone Oct 16, 2019
@bbondy
Copy link
Member

bbondy commented Oct 16, 2019

FYI I'm going to closet his out with this pull request:
brave/ethereum-remote-client#85

But I'm sure there will be more translation issues. Those should be tracked in separate more specific issues please and thanks.

@NumDeP
Copy link

NumDeP commented Oct 17, 2019

@ryanml I didn't know whether the bit above congratulations was a translation issue but it's typically what shows up when certain foreign languages aren't supported in brave. In addition, I don't expect to remove and re-create the wallet again so there wouldn't be a way for me to check if it was after the next couple of updates, thus didn't want it to be missed. If it is, following bondy's comment perhaps you can make a specific issue regarding it.

Screenshot_20191017_092158

@srirambv
Copy link
Contributor Author

@NumDeP can you please log a new issue for it and mention the OS details. The icon is different on different platforms.

@srirambv
Copy link
Contributor Author

Moving the issue out of 1.0.11 release as only part of the text is translated. Follow up issue #6533 covers all untranslated text for new custom texts that needs to be verified on the next release

@srirambv srirambv removed this from the Ethereum Remote Client 1.0.11 milestone Oct 18, 2019
ryanml pushed a commit to brave/ethereum-remote-client that referenced this issue Dec 5, 2019
@bbondy bbondy added this to the Closed / Invalid milestone Jun 3, 2020
@srirambv srirambv added feature/ethereum-remote-client and removed feature/web3/wallet Integrating Ethereum+ wallet support labels Sep 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment