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

Remove contact page section and disable crowdin as translation tool #411

Merged
merged 4 commits into from
Feb 19, 2024

Conversation

iuricmp
Copy link
Contributor

@iuricmp iuricmp commented Feb 9, 2024

This pull request disables crowdin as the translation tool.


We encourage you to read our dedicated [blog post](https://berty.tech/blog/berty-translation/)
We encourage you to open a new PR on the [berty](https://github.com/berty/berty) repository.
All the translations are stored in the [/i18n/locale](https://github.com/berty/berty/tree/master/js/packages/i18n/locale) folder.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is for contributing translations to the Berty app. Do you want to add links for contributing translations to the Berty web site? I ask because we do already have translations: https://berty.tech/pt

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've included a simple website mention in this commit chore: translation instruction for website

@jefft0 jefft0 mentioned this pull request Feb 16, 2024
1 task
@jefft0
Copy link
Contributor

jefft0 commented Feb 16, 2024

Makes sense. @iuricmp will update the message in multiple places.

@iuricmp iuricmp requested a review from jefft0 February 16, 2024 15:26
Copy link
Contributor

@jefft0 jefft0 left a comment

Choose a reason for hiding this comment

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

LGTM

@iuricmp iuricmp merged commit b8319ae into berty:master Feb 19, 2024
6 checks passed
@iuricmp iuricmp deleted the 408-remove-crowdin-for-now branch February 19, 2024 08:32
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.

2 participants