openownership / openownership.org-wagtail

Open Ownership Wagtail site.
Other
1 stars 1 forks source link

Feb 2023 translation changes #310

Closed StephenAbbott closed 1 year ago

StephenAbbott commented 1 year ago

Hi @drcongo. This pull request is to override some of the automatic language translations that Wagtail is doing to some of the site furniture which my comms colleagues want to undo.

Question for you though: we've noticed that the purple 'Language' button appearing next to the cover images on publications like this one are also getting translated in some cases. So in this example, Langue is appearing instead of Language. We'd like it to always appear as 'Language' for all languages but I can't see that field in the .po files.

Is there another file which could be tweaked so that 'Language' shows for all locales/languages?

drcongo commented 1 year ago

The Langue button change I can make, I don't think it's possible to use the .po files to say "don't translate this" but I can definitely do that in the template very easily.

StephenAbbott commented 1 year ago

@drcongo Language button still appears to be getting auto-translated if you shift to, for example, Spanish translation https://www.openownership.org/es/publicaciones/datos-estructurados-e-interoperables-de-beneficiarios-finales/

image

Changes to the es .po file - see https://github.com/openownership/openownership.org-wagtail/pull/310/commits/aebe3e5a0c2ae68380bd1419ccc1fcced1e28a08 - are also not being displayed on the production site. For example, 'Buscar' showing in main navbar instead of 'Register'.

Is there a further step we need to take to force Wagtail to use these rather than the automatic translations?

StephenAbbott commented 1 year ago

@drcongo Have these changes been deployed to production? I think we need to revisit them if so as the default Wagtail translations are still showing up on the live site