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

Transifex translations are not being added during a release #441

Open
bitwingsde opened this issue Nov 29, 2021 · 5 comments
Open

Transifex translations are not being added during a release #441

bitwingsde opened this issue Nov 29, 2021 · 5 comments
Labels
impact/medium type/bug Something isn't working

Comments

@bitwingsde
Copy link

bitwingsde commented Nov 29, 2021

There is a German translation for this module in transifex, but the translations are not being pulled when the module gets new minor releases:

https://www.transifex.com/silverstripe/silverstripe-mfa/language/de/

@GuySartorelli GuySartorelli changed the title Add transifex translations Transifex translations are not being added by COW Jun 21, 2022
@GuySartorelli GuySartorelli changed the title Transifex translations are not being added by COW Transifex translations are not being added during a release Jun 21, 2022
@GuySartorelli GuySartorelli added type/bug Something isn't working impact/medium labels Jun 21, 2022
@tom890180
Copy link

Hey,

We've translated this module to swedish and finnish as well, and we'd appreciate if these translations were pulled in. I would make a pull request myself but on tx pull I don't get any new ones (not even the german ones), so I assume they need to be reviewed first (?)

Tom

@NightJar
Copy link
Contributor

NightJar commented Nov 2, 2022

@GuySartorelli Thanks for updating the translations. The OP here from @bitwingsde mentions that this has been an issue over several releases in the past though. Is there an issue with the release process that needs to be fixed?

@GuySartorelli
Copy link
Member

@emteknetnz was the person who updated the translations and has done releases before whereas I have only supported.
To the best of my knowledge the release process will pull in translations, I can't speak for why this might not have happened in previous releases.

@emteknetnz
Copy link
Member

I'm not 100% sure, though my best guess was there wasn't a minor release for the MFA module during the previous minor release (4.11), thus the translations were not run. At this stage translations are only regularly run and released during "official" minor releases where the sub modules (such as the MFA module) also have new minor releases created.

@NightJar
Copy link
Contributor

NightJar commented Nov 3, 2022

Yeah, the translation update is run as part of the "main" release tool, cow. Minor releases tagged or created via GitHub won't have that step taken automatically and must be performed manually beforehand.

I thought cow checked for translations during the release process so would create a releasable commit "on the fly" during the process... but I'm probably remembering wrong & the tx update happens after the "should release" calculation.

Anyway... is there some kind of "before release" GitHub Action hook that could do a thing like that, so it isn't forgotten in the future? And/or pre-tag git hook?

Just brainstorming ideas at this point to try and avoid this in the future :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/medium type/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants