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

Automated integration with Crowdin #1562

Open
ludekjanda opened this issue Sep 8, 2023 · 7 comments
Open

Automated integration with Crowdin #1562

ludekjanda opened this issue Sep 8, 2023 · 7 comments
Labels
i18n translation https://github.com/letsencrypt/website/blob/main/TRANSLATION.md

Comments

@ludekjanda
Copy link
Collaborator

Crowdin supports automated syncronization with GitHub repositories:

https://support.crowdin.com/github-integration/

Advantages:

  • automated source refresh in Crowdin
  • new files are automatically added to Crowdin
  • export back to GitHub is also automated (only 100% proofread files would be exported)

Prerequisites:

@ludekjanda ludekjanda added translation https://github.com/letsencrypt/website/blob/main/TRANSLATION.md i18n labels Sep 8, 2023
@ludekjanda
Copy link
Collaborator Author

Here is a nice video how to integrate using GitHub actions: https://www.youtube.com/watch?v=5b7BMuCoKGg I have done several integrations in the past, I could help with that.

@tdelmas
Copy link
Collaborator

tdelmas commented Sep 8, 2023

The problem was that a few files generated by Crowdin were invalid (partially because of macros, partially because they convert markdown to html then to sentences for translation, then reverse, which may generate a few odd artefacts/bugs). It may be fixed now, we need to check if we can import without manual intervention.

@ludekjanda
Copy link
Collaborator Author

@tdelmas we need to verify whether these bugs are solved now. I am not sure what is the difference between the automated sync and the manual download of translation. Does it mean that on manual download, some strings need some extra manual intervention? I am in close contact with the Crowdin team, such bugs can be fixed easily.

Please can you also clarify the whole process? Where can I find the latest sources? Here? https://github.com/letsencrypt/website/tree/main/content/en

Some files are missing in Crowdin now, for example this one: https://github.com/letsencrypt/website/blob/main/content/en/docs/too-many-registrations-for-this-ip.md

I think I need to refresh sources in Crowdin now. For the next steps, I could set up a paralel project for testing and link it to my forked repo. We could test whether the automatically downloaded files have any errors on pull request...

What do you think?

@ludekjanda
Copy link
Collaborator Author

@tdelmas sorry I am overcomplicating. We do not need setting up a parallel process just for testing it.
As the automated translations are being output into a separate l10n branch, not into the main, we can safely set up the whole integration and test the result when comparing both sites from both branches. The only change in the main branch would be that extra file crowdin.yaml added there and 2 actions.

@bdaehlie
Copy link
Collaborator

I don't understand the Crowdin side of things well but I added the GitHub app to Crowdin. It is not set up, do you have privileges to do that @ludekjanda ?

@ludekjanda
Copy link
Collaborator Author

@bdaehlie thanks Josh, I can see it in Crowdin now. I just did a few click through it, it seems I should be able to set it up. Let's see. My priority is to do the first push manually to see all problems that may arise with formatting. I'll need @tdelmas help with verifying, probably.

@tdelmas
Copy link
Collaborator

tdelmas commented Sep 15, 2023

@ludekjanda do not hesitate to ping me when you need!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n translation https://github.com/letsencrypt/website/blob/main/TRANSLATION.md
Projects
None yet
Development

No branches or pull requests

3 participants