Skip to content

fix(settings): use correct scope for translations - #53695

Merged
kesselb merged 1 commit into
masterfrom
fix/settings-l10n
Jun 26, 2025
Merged

fix(settings): use correct scope for translations#53695
kesselb merged 1 commit into
masterfrom
fix/settings-l10n

Conversation

@susnux

Copy link
Copy Markdown
Contributor

Summary

The translations are in settings not in core so the scope was set wrong.

Checklist

@susnuxsusnux added this to the Nextcloud 32 milestone Jun 26, 2025
@susnux
susnux requested a review from a team as a code ownerJune 26, 2025 10:41
@susnux
susnux requested review from Altahrim, icewind1991 and sorbaugh and removed request for a teamJune 26, 2025 10:41
@susnuxsusnux added bug 3. to review Waiting for reviews feature: language/translations (l10n/i18n) Localization and translation matters labels Jun 26, 2025
@susnux

Copy link
Copy Markdown
ContributorAuthor

/backport to stable31

@susnux
susnux requested review from come-nc and removed request for sorbaughJune 26, 2025 10:41

@come-nccome-nc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can use DI and you will get a scoped IL10N instance because you’re in OCA.

@susnux
susnuxforce-pushed the fix/settings-l10n branch from 973a735 to 8d21c68CompareJune 26, 2025 16:08
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnuxforce-pushed the fix/settings-l10n branch from 8d21c68 to b1282f7CompareJune 26, 2025 16:26
@kesselb
kesselb merged commit 0efd05a into masterJun 26, 2025
@kesselb
kesselb deleted the fix/settings-l10n branch June 26, 2025 18:09
@susnux

Copy link
Copy Markdown
ContributorAuthor

/backport to stable30

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: language/translations (l10n/i18n)Localization and translation matters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: No translation in MimeTypeMigrationAvailable class

6 participants

@susnux@icewind1991@sorbaugh@come-nc@kesselb@skjnldsv