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

New Crowdin translations by GitHub Action #1617

Merged
merged 46 commits into from
Feb 5, 2025
Merged

Conversation

joomla-translation-bot
Copy link
Collaborator

No description provided.

@joomla-translation-bot joomla-translation-bot force-pushed the l10n_crowdin_package_v5 branch 5 times, most recently from 038b785 to 30f8055 Compare February 5, 2025 04:04
@tecpromotion
Copy link
Contributor

@zero-24 @heelc29 @HLeithner @wilsonge any idea why COM_GUIDEDTOURS_TOUR_GUIDEDTOURSTEPS_STEP_TARGET_DESCRIPTION was deleted in all translations in crowdin? I can still see the source string:
Bildschirmfoto 2025-02-05 um 09 02 12

The relevant core PR was joomla/joomla-cms#44793

@zero-24
Copy link
Contributor

zero-24 commented Feb 5, 2025

Hmm it does not sound sensible to me maybe its worth reaching out to the crowdin support?

@tecpromotion
Copy link
Contributor

Hmm it does not sound sensible to me maybe its worth reaching out to the crowdin support?

Yes, that would be my next step.

@heelc29
Copy link
Contributor

heelc29 commented Feb 5, 2025

I think it is correct. The string has been updated, even if only one dot.
Since we only download approved translations and this one has not been approved yet, it has been removed.

@tecpromotion
Copy link
Contributor

I think it is correct. The string has been updated, even if only one dot. Since we only download approved translations and this one has not been approved yet, it has been removed.

This happens quite often and usually an existing string is not simply deleted in Crowdin when something changes in the source. That would be new to me. I have now restored this for ‘Basque’ in crowdin and restarted the action.

@tecpromotion
Copy link
Contributor

I think it is correct. The string has been updated, even if only one dot. Since we only download approved translations and this one has not been approved yet, it has been removed.

This happens quite often and usually an existing string is not simply deleted in Crowdin when something changes in the source. That would be new to me. I have now restored this for ‘Basque’ in crowdin and restarted the action.

@heelc29 you're probably right and I've never really noticed it before.

@tecpromotion tecpromotion merged commit 869ba30 into main Feb 5, 2025
11 checks passed
@tecpromotion tecpromotion deleted the l10n_crowdin_package_v5 branch February 5, 2025 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants