-
Notifications
You must be signed in to change notification settings - Fork 195
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
Swap "accounts" for "wallets" copy #2872
Comments
@stackingsaunter Could you please provide path details of the files that need to be changed? It would be really helpful. |
@ayushdhiman-py this is part of the task to figure that out and make the changes. |
@stackingsaunter I went through the figma files and reviewed all the required changes, I will immediately start working on the issue once I finish setting it up. |
@stackingsaunter sorry but i won't be continuing the issue, thank you as I got to learn about react-i18n. Also this was my first time working on an extension. |
@Harshit-Prasad thanks for you work. I don't think the translations are an issue here and we can remove the outdated keys of the other languages by running a script. |
@rolznz I'll be taking on this issue. |
@AdarshRawat1 forgot to mention this in initial post On Wallet Settings page in "Danger" zone copy also must be updated to wallets: |
@stackingsaunter noted !! |
@stackingsaunter, if this issue is still valid, I would like to work on it. I noticed that in the wallet setting screen, we still use the account name in some places, such as:
|
@amitamrutiya2210 Good point, I just revisited the translations file and found 50 mentions of |
Part of #2871 epic, which full desired effect can be viewed in this Figma
Goal is to change copy related to "extension accounts" that from now on will be called "wallets".
List of changes:
Dropdowns
"Settings" page
"Account settings" page
"Accounts" (Manage accounts)
The text was updated successfully, but these errors were encountered: