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

add inlang to make the contribution of translations easier #1377

Open
wants to merge 156 commits into
base: master
Choose a base branch
from

Conversation

felixhaeberle
Copy link

Let me know what changes you request for merging this PR.

Description

This pull request adds the possibility for contributors and translators to manage translations in a UI instead of files with no overhead for the maintainers. Additionally, I fixed some translations and added the German (de) language.

To get a UI for translations contained in this repository, an inlang.config.js has been created at the root of the repository. Furthermore, I added the few missing translations. If you want, I can add a small i18n section to the documentation on the README.

Overview

The changes of this PR and a live instance of the editor can be previewed with the following link https://inlang.com/editor/github.com/felixhaeberle/localtuya.
Note: This link should be changed to point to rospogrigio instead of felixhaeberle after this PR has been merged.

Limitations

Certain actions are slow
Inlang is running entirely on git, giving tremendous CI/CD and contribution power to localization. That means that the whole localtuya repo is cloned into the browser which makes certain actions like the initial load and pushing changes slow. Those limitations will be fixed with future releases and require no input from localtuya.

232016041-ec0c3da3-94a9-4492-85dd-d8478c66d945 (1)

Preview

Preview the messages on https://inlang.com/editor/github.com/felixhaeberle/localtuya.

@felixhaeberle
Copy link
Author

@rospogrigio Hey, just wanted to check-in if you want to improve the localization in your project? Let me know how I can further assist ✨

@rospogrigio
Copy link
Owner

I really like this PR, can you document it in the README.md and info.md and as you proposed?
Thank you for the contribution!

@felixhaeberle
Copy link
Author

@rospogrigio Thank you, will do a follow-up PR with the info 👍

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