- Issue created by @van.dordafog
this patch just renames the klaro.texts app to service wherever it's used in code and adds an update hook to change the existing config.
might have issues on sites where those texts where never translated in BE (not tested)- π©πͺGermany sascha_meissner Planet earth
Hey Van.dordafog thank you for your support!
This might be related to https://www.drupal.org/project/klaro/issues/3161254 π Text translations not working properly Active
is BE also the default language on your site?IΒ΄m going to test it though, iΒ΄m not sure if renaming app to service is sufficient fix to merge
@sascha_meissner the easiest way to test this would be to change the config text in "Texts > App Item > PURPOSE > Singular form" to something noticeably different. Then check in frontend list of apps - the "purpose" text under each app will not have changed.
- π©πͺGermany sascha_meissner Planet earth
The latest klaro release (rc6) included some improvements related to translations,
could you maybe check if you can reproduce your issue with a fresh install? - π©πͺGermany jan kellermann
jan kellermann β made their first commit to this issueβs fork.
- Merge request !22Resolve #3403658: Fixed bug for changing klaro texts. β (Merged) created by jan kellermann
- π©πͺGermany jan kellermann
Thank you very mich. I tested, added a changelog entry and created a MR.
I tested and reviewed. It would be nice if someone else could do a review.
- π©πͺGermany jan kellermann
Tested with current dev version. Thank you @van.dordafog!
-
jan kellermann β
committed 7a43e5cb on 3.x
Resolve #3403658: Fixed bug for changing klaro texts.
-
jan kellermann β
committed 7a43e5cb on 3.x