Skip to content

Conversation

@meow-meooow
Copy link

@meow-meooow meow-meooow commented Apr 5, 2025

  • Update translations for social login, wallet, network, etc.

Motivation and Context

Some of the Japanese sentences were strange and would cause distrust among users of the wallet (our service as well).

My changes should be reflected in the areas underlined in the image.

PACKS _ Home - Google Chrome 2025_04_05 20_45_12
Screenshot 2025_04_05 20_47_04

Description

I have likewise corrected the apparently incorrect wording that I found in the course of searching for the relevant section.

And in the process of creating this PR, I noticed that the “一般的な” at the top of the second image is also clearly wrong. It should be "一般".

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project. (run lint)
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • My code requires a db migration.

- Update translations for social login, wallet, network, etc.
Copy link
Member

@shahbaz17 shahbaz17 left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution, @meow-meooow

Keep this coming, appreciate the detailed look at these translations.

},
"footer.message-new": {
"dutch": "Zelfkusten via",
"english": "Self-custody via",

Choose a reason for hiding this comment

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

@shahbaz17 Can you tell me what this full string might say, please? The Japanese linguist is confused what would come after "Self-custody via..". Thank you!

Copy link
Member

Choose a reason for hiding this comment

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

This is used in the previous versions to show

"Self-custody via Web3Auth"

Image

},
"social.continue": {
"dutch": "Doorgaan met",
"english": "Continue with",

Choose a reason for hiding this comment

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

@shahbaz17 Can you also please help me with more context for this string?

Copy link
Member

Choose a reason for hiding this comment

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

This is Continue with {login provider name}

Image

@i18nlaurel
Copy link

Thank you @shahbaz17! I passed this along to our linguist team.

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.

4 participants