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

Many bindings not available in Crowdin #10635

Closed
lolodomo opened this issue May 2, 2021 · 8 comments
Closed

Many bindings not available in Crowdin #10635

lolodomo opened this issue May 2, 2021 · 8 comments
Labels
bug An unexpected problem or unintended behavior of an add-on i18n

Comments

@lolodomo
Copy link
Contributor

lolodomo commented May 2, 2021

Translations for the Netatmo binding is not available in Crowdin.

@cweitkamp : any idea why ? Could it be because there is no netatmo.propertioes file in the binding ?

PS: I volunteer to approve French translations in the openhab-addons repo

@lolodomo lolodomo added the bug An unexpected problem or unintended behavior of an add-on label May 2, 2021
@lolodomo
Copy link
Contributor Author

lolodomo commented May 2, 2021

In fact, it looks like a lot of bindings are missing in Crowdin.
In French translations, there are 42 bindings in Crowdin.
Countining the projects having i18n folder with properties files: around 105
As another example, my sonyprojector binding is missing too.

@lolodomo lolodomo changed the title [netatmo] Netatmo binding not available in Crowdin Many bindings not available in Crowdin May 2, 2021
@cweitkamp
Copy link
Contributor

@lolodomo Yes, exactly. To be recognized and uploaded to Crowdin the bundle needs one or more *.properties files in the OH-INF/i18n/ folder. Once it has been placed there it will be synchronized.

Currently I am working on a XSLT transformation to automatically generate properties files from our XML definitions. I will commit a draft later.

Thanks for volunteering. I granted Proofread permissions to you on Crowdin for French.

@lolodomo
Copy link
Contributor Author

lolodomo commented May 2, 2021

Currently I am working on a XSLT transformation to automatically generate properties files from our XML definitions. I will commit a draft later.

Just for my understanding: why now do we have certain bindings but not others ? They were added manually in Crowdin ?

@cweitkamp
Copy link
Contributor

Crowdin integration looks for an existing *.properties file to synchronize it. Property files having a location suffix are excluded. That why only part of them are already available for translation.

@cweitkamp
Copy link
Contributor

See #10636 for a draft of the XSLT transformation.

@cweitkamp cweitkamp added the i18n label May 2, 2021
@clinique
Copy link
Contributor

I second @lolodomo on this. My dev time is a bit restricted currently, so I can help working on translations, and I was also looking for Netatmo binding in Crowdin :-).

@cweitkamp
Copy link
Contributor

cweitkamp commented May 29, 2021

@clinique Thanks you for offering to help translating add-ons. If you like to add translations for Netatmo someone has to create the properties fuel containing all English source strings and commit it to this repository. After that they will be uploaded automatically and available on Crowdin. Same for other add-ons too.

@cweitkamp
Copy link
Contributor

Closed by #11760 (and others).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on i18n
Projects
None yet
3 participants