tdelmas / website

Let's Encrypt Website and Documentation
https://letsencrypt.org
Mozilla Public License 2.0
2 stars 3 forks source link

Crowdin #45

Open tdelmas opened 4 years ago

tdelmas commented 4 years ago

Github integration: blocked by a bug. TODO: report that bug.

Before Github integration: Copy sentence by sentence existing translations into crowdin. (Time consuming!) Check export New translations can happen Export translations and fix bug

When the Github bug is resolved: Set up Github integration Use TM (translation memory) to copy translations to synced files, remove old not synced files Use pr to integrate translations

tdelmas commented 4 years ago

bug with link reported

new bug in _index discovered

tdelmas commented 4 years ago

it does rewrite the html and markdown links

tdelmas commented 4 years ago

bug with link AND relref

tdelmas commented 4 years ago

Hi,

Sorry to bother you, do you have any update about that issue?

Where the macro: {{<link "Certificate Transparency log" "/docs/ct-logs" >}}

Appears as:{{<link "Certificate Transparency log" ">}}

I tried to change the definition of the macro to: {{}}

But the result after download is: {{

}}

which doesn't work: Hugo macro starts with "{{<" without space and ends with ">}}" without space nor "/".

If you think the bug will be fixed in a short time, I can wait, if the bug is more complicated to fix (or the current situation make it difficult for you to do so), I probably can use the new macro format with an additional script to fix the file, but it will make github integration impossible, I'll have to revert all those changes once the bug is fixed.

That's why if you could provide an estimate it would be valuable for me

To be sure there is no other problem I tested all pages and discovered another bug:

I have a .md file that starts with the following content:


title: Documentation top_graphic: 1 menu: "main" menu: main: weight: 10

And when the translation is downloaded the file starts with:

title: Documentation top_graphic: 1 menu: main: weight: 10

Note the missing "menu" line.

That bug is not a big deal (it only prevent GitHub integration), and is only present in one of the file, so we can fix that manually, but I felt it was important to notify you You can find in attachment a screenshot of that file, the version uploaded and the version downloaded.

I know those are difficult times for everybody, and hope you're well. Thank you again for your time, Best regards Tom

tdelmas commented 4 years ago

image

and relref in docs/ct-logs.html and sponsors.html

comradekingu commented 4 years ago

Please don't use Crowdin. Weblate is libre software and it works for the user, as opposed to sublicensing strings and running every spying script under the sun.

tdelmas commented 4 years ago

That decision has already be made, and I think it was the right one Yes, I'm disappointed that Crowdin in not FOSS, but it was the best option to translate full documents where partial updates are made over time. Weblate is good to translated individual sentence without context