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

fix(theming): use plain string in t method (l10n) #51266

Merged
merged 2 commits into from
Mar 5, 2025
Merged

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Mar 5, 2025

Summary

To correctly extract the translations we only can have plain strings within the t method.
Also applied new l10n feature to move link code into translation function cleaning up a bit.

Checklist

@susnux susnux added this to the Nextcloud 32 milestone Mar 5, 2025
@susnux susnux requested a review from a team as a code owner March 5, 2025 08:47
@susnux susnux requested review from artonge, skjnldsv and sorbaugh and removed request for a team March 5, 2025 08:47
@susnux susnux force-pushed the fix/l10n-plain-string branch from 51a20c8 to 74baabd Compare March 5, 2025 08:48
@susnux susnux force-pushed the fix/l10n-plain-string branch from 74baabd to 6c5f246 Compare March 5, 2025 15:38
@susnux
Copy link
Contributor Author

susnux commented Mar 5, 2025

/compile

@susnux susnux enabled auto-merge March 5, 2025 15:38
To correctly extract the translations we only can have plain strings
within the `t` method.
Also applied new l10n feature to move link code into translation
function cleaning up a bit.

Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux force-pushed the fix/l10n-plain-string branch from 6c5f246 to c0dcac0 Compare March 5, 2025 15:39
Signed-off-by: nextcloud-command <[email protected]>
@susnux susnux merged commit c1d8795 into master Mar 5, 2025
121 checks passed
@susnux susnux deleted the fix/l10n-plain-string branch March 5, 2025 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

chore(i18n): leading dot needs to be removed
4 participants