Skip to content

Conversation

@devfelipenunes
Copy link
Contributor

@devfelipenunes devfelipenunes commented Dec 10, 2024

This pull request adds the complete Portuguese (Brazil) translation for the wallet connection module. It includes translations for connecting a wallet, sending/receiving funds, network management, and other wallet-related actions. This improves the user experience for Portuguese-speaking users.


PR-Codex overview

This PR introduces the complete Portuguese (Brazil) translation for the wallet connection module, enhancing the user experience for Portuguese-speaking users by providing translations for various wallet-related actions.

Detailed summary

  • Added "pt_BR" to the locale types.
  • Implemented Portuguese translations in various files for wallet connection, injected wallet, and smart wallet functionalities.
  • Created br.ts files with localized strings for ConnectLocale, InjectedWalletLocale, SmartWalletLocale, and InAppWalletLocale.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@changeset-bot
Copy link

changeset-bot bot commented Dec 10, 2024

🦋 Changeset detected

Latest commit: b515649

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
thirdweb Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Dec 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
thirdweb_playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 11, 2024 0:55am

@graphite-app
Copy link
Contributor

graphite-app bot commented Dec 10, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@vercel
Copy link

vercel bot commented Dec 10, 2024

@devfelipenunes is attempting to deploy a commit to the thirdweb Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added packages SDK Involves changes to the thirdweb SDK labels Dec 10, 2024
@joaquim-verges
Copy link
Member

Thank you @devfelipenunes ! there's a couple more files that contain strings to translate, could you look into translating thos too? look for all the en.ts files in the repo, you'll see there's different ones

- Translated `connectWalletLocale` to Brazilian Portuguese.
- Translated `inAppWalletLocale` to Brazilian Portuguese.
- Translated `injectedWalletLocale` to Brazilian Portuguese.
- Translated `smartWalletLocale` to Brazilian Portuguese.

These changes provide support for Brazilian Portuguese users across multiple wallet-related locales, improving the application's accessibility and user experience.
@devfelipenunes
Copy link
Contributor Author

This PR adds Brazilian Portuguese translations for the following locales:

  • connectWalletLocale
  • inAppWalletLocale
  • injectedWalletLocale
  • smartWalletLocale

@joaquim-verges, could you please review these changes to ensure the translations align with the project's requirements? Let me know if any adjustments are needed. Thanks!

@devfelipenunes
Copy link
Contributor Author

@joaquim-verges Can you help me? I'm having trouble merging.

@joaquim-verges joaquim-verges merged commit 8020bdb into thirdweb-dev:main Dec 11, 2024
22 of 27 checks passed
@joaquim-verges
Copy link
Member

@devfelipenunes merged! thanks again!

@jnsdls jnsdls mentioned this pull request Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packages SDK Involves changes to the thirdweb SDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants