Skip to content

Conversation

@SebastianKrupinski
Copy link
Contributor

  • Resolves: # Client ticket

Summary

  • Changed translation source to files app

Checklist

@SebastianKrupinski SebastianKrupinski changed the title fix:remote account activity translation fix: remote account activity translation Mar 4, 2025
@SebastianKrupinski SebastianKrupinski self-assigned this Mar 4, 2025
@SebastianKrupinski SebastianKrupinski added the 3. to review Waiting for reviews label Mar 4, 2025
if (!isset($parsedParameters['user'])) {
// External user via public link share
$subject = str_replace('{user}', $this->activityLang->t('"remote account"'), $subject);
$subject = str_replace('{user}', $this->l->t('"remote account"'), $subject);
Copy link
Member

Choose a reason for hiding this comment

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

@nickvergessen could you please check if this is fine? activityLang was introduced in #4969

Copy link
Member

Choose a reason for hiding this comment

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

Should be fine, I guess (2016) we tried to avoid translating it twice.

Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

Looks good but didn't test

@SebastianKrupinski
Copy link
Contributor Author

/backport to stable31

@SebastianKrupinski
Copy link
Contributor Author

/backport to stable30

@SebastianKrupinski
Copy link
Contributor Author

/backport to stable29

Copy link
Member

@st3iny st3iny left a comment

Choose a reason for hiding this comment

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

LGTM!

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 5, 2025
Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
@SebastianKrupinski SebastianKrupinski force-pushed the fix/noid/remote-account-activity-translation branch from c6272de to 45a49e8 Compare March 7, 2025 14:23
@SebastianKrupinski SebastianKrupinski requested a review from a team as a code owner March 7, 2025 14:23
@SebastianKrupinski SebastianKrupinski requested review from ArtificialOwl, artonge and sorbaugh and removed request for a team March 7, 2025 14:23
@SebastianKrupinski SebastianKrupinski merged commit 49e52c1 into master Mar 7, 2025
195 of 204 checks passed
@SebastianKrupinski SebastianKrupinski deleted the fix/noid/remote-account-activity-translation branch March 7, 2025 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants