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

Translate ENS Dapp! #76

Open
chadyj opened this issue Oct 26, 2018 · 27 comments
Open

Translate ENS Dapp! #76

chadyj opened this issue Oct 26, 2018 · 27 comments
Labels

Comments

@chadyj
Copy link

chadyj commented Oct 26, 2018

We have users all over the world. Let's give them the ENS dapp in their local language. 🇺🇸 🇨🇳 🇫🇷 🇯🇵

The mobile app uses Lokalise to manage the translation of strings and we have a nice UI to manage the translations then a git-based workflow to commit and review changes.

The mobile app uses react-native-i18n, but for this dapp you can use one of these supported file formats https://docs.lokalise.co/supported-file-formats such as json

Once there is a compatible strings file I'd be happy to help setup things on the Lokalise side. Just ping me :)

@rachelhamlin
Copy link

Adding a bounty to this one.

The ask is to format all the ENS DApp strings into a Lokalise compatible file, e.g. JSON. (cc @bgits for any support)

@chadyj
Copy link
Author

chadyj commented Nov 20, 2018

Another example is status.im using YAML over here https://github.com/status-im/status.im/tree/develop/themes/navy/languages

Perhaps @PierrickGT might be interested who just did some translation work at status.im?

@PierrickGT
Copy link
Contributor

@chadyj yes, I am interested by this task. I've already done some similar work for Balance.

You can see what I've worked on in the following PR:
balance-io/balance-wallet#103
balance-io/balance-common#6
balance-io/balance-manager#315

@chadyj
Copy link
Author

chadyj commented Nov 22, 2018

Sweet @PierrickGT! Good luck! Ping me when you have the EN language file and I'll add to Lokalise then get translators gong :)

We also need a way to detect and set the language, and perhaps a way to let the user change the language. WDYT @rachelhamlin ?

@PierrickGT
Copy link
Contributor

@chadyj will you add a bounty on Gitcoin in the coming days?

@rachelhamlin
Copy link

@PierrickGT can you handle device-based language detection as part of this bounty? I'm bumping it up to a medium.

@andytudhope will help us get this funded!

@PierrickGT
Copy link
Contributor

@rachelhamlin yes, I can handle device-based language detection with i18n-js.

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


This issue now has a funding of 300.0 DAI (300.0 USD @ $1.0/DAI) attached to it as part of the Status.im fund.

@gitcoinbot
Copy link

gitcoinbot commented Nov 22, 2018

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work has been started.

These users each claimed they can complete the work by 3 years, 2 months ago.
Please review their action plans below:

1) lorissikora has applied to start work (Funders only: approve worker | reject worker).

I‘m intetested. Could translate to german.
2) pierrickgt has been approved to start work.

I'll first setup i18n-js to detect and load which language should be use depending on the device language.
I will then replace text strings in the app by the i18n function to translate the string in the appropriate language.

Learn more on the Gitcoin Issue Details page.

@gitcoinbot
Copy link

@PierrickGT Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@PierrickGT
Copy link
Contributor

PierrickGT commented Nov 26, 2018

@chadyj @bgits how can I test the app in local development?
Cause I'm connected on the Ropsten network through Metamask but after clicking on the Let's Go button, I get this error message:
Please connect to Ethereum Development to continue.

I'm connected to the local blockchain running but I've got an error in the console saying that the connection to the endpoint http://localhost:5001/api/v0/id?stream-channels=true is refused.

@rachelhamlin
Copy link

@PierrickGT could be an issue with browser privacy mode. What happens if you toggle that off from within the profile?

@PierrickGT
Copy link
Contributor

@rachelhamlin you mean in my browser or in the app? Cause I don't see any profile section in the app.

@bgits
Copy link
Contributor

bgits commented Nov 28, 2018 via email

@PierrickGT
Copy link
Contributor

A different version needs to be launched for Ropsten. If your developing locally you can run the command embark run testnet

On Mon, Nov 26, 2018 at 5:30 PM Pierrick TURELIER @.***> wrote: @chadyj <@chadyj> @bgits <@bgits> how can I test the app in local development? Cause connected on the Ropsten network through Metamask but after clicking on the Let's Go button, I get this error message: Please connect to Ethereum Development to continue. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#76 (comment)>, or mute the thread <github.com/notifications/unsubscribe-auth/AD1YSKgxyxSkcJS9Pgikq-qUJvFNiNp3ks5uzGtpgaJpZM4X8jy-> .

Still no luck with embark run testnet.

@PierrickGT
Copy link
Contributor

@chadyj @bgits @rachelhamlin I've finished to add the translation on all the pages.
I'm awaiting your review approval before sending my work through Gitcoin.

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work for 300.0 DAI (300.0 USD @ $1.0/DAI) has been submitted by:

  1. @pierrickgt

@StatusSceptre please take a look at the submitted work:


@chadyj
Copy link
Author

chadyj commented Dec 4, 2018

Thank you @PierrickGT! The changes have been merged and translation files imported into Lokalise.

Next, we need contributors to translate the 471 words.

We have @jinhojang93 to do Korean.
@PierrickGT already did French.

Would love to get Chinese, Farsi, and Russian too. Actually all languages sound good!

@pablanopete @j-zerah Can you help get some translators together? (or alternatively we can pay for it via a service for a quick turn around)

@rachelhamlin
Copy link

Thanks so much @PierrickGT and @chadyj for managing. What's the best way to filter for the ENS strings on Lokalise?

We have someone who can translate to Chinese here in China.

@chadyj
Copy link
Author

chadyj commented Dec 4, 2018

Thanks so much @PierrickGT and @chadyj for managing. What's the best way to filter for the ENS strings on Lokalise?

We have someone who can translate to Chinese here in China.

Sweet!

AENS DApp project is setup with just the ENS strings. Its ready to go :)

@rachelhamlin
Copy link

@chadyj got it. Will share with our Chinese community member :)

@jinhojang6
Copy link

Hi all, can I get the context or screens of these strings?

  1. Nice!
  2. Of
  3. Ethereum address domain resolves to
  4. Status messenger address domain resolves to
  5. Read your account token balance
  6. Read account token balance

@chadyj
Copy link
Author

chadyj commented Dec 4, 2018

Thanks @jinhojang93! Super fast work :)

Hi all, can I get the context or screens of these strings?

@bgits @3esmit @rachelhamlin Got screenshots of these? I couldn't find it.

btw I uploaded several other screenshots and uploaded to Lokalise for translating context.

@gitcoinbot
Copy link

⚡️ A *Javascript Master Scriber* Kudos has been sent to @PierrickGT for this issue from @StatusSceptre. ⚡️

Nice work @PierrickGT!
Your Kudos has automatically been sent in the ETH address we have on file.

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


The funding of 300.0 DAI (300.0 USD @ $1.0/DAI) attached to this issue has been approved & issued to @PierrickGT.

@rachelhamlin
Copy link

The DApp shows extra content if you load it from a normal browser using MetaMask, e.g.

image

Would be great to ultimately remove that content & make it consistent with the mobile version, but @bgits do you recognize these particular strings from @jinhojang93?

@TouchstoneTheDev
Copy link

TouchstoneTheDev commented Jan 27, 2024

Hi
I would like to translate the strings, I have previous experience also !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants