Skip to content

Comments

fix: translation for recently contacted not working#4662

Merged
kesselb merged 1 commit intomainfrom
fix/translation-for-recently
Aug 29, 2025
Merged

fix: translation for recently contacted not working#4662
kesselb merged 1 commit intomainfrom
fix/translation-for-recently

Conversation

@kesselb
Copy link
Contributor

@kesselb kesselb commented Aug 28, 2025

I came across this issue while working on a related case with a user who was using Nextcloud in different languages.

The contactsinteraction translations are not loaded in the Contacts app, so the source string is never translated. It’s just a detail, but it would look nicer if the menu item were shown correctly.

How to reproduce:

  • Use Nextcloud in a non-English language
  • Share a file with [email protected]
  • The "recently contacted" list is displayed as a contact list, but it should appear in the top navigation like it does in English

Note:

There will be a follow-up for contacts and server removing the translation, but this will take some more time. As this change is small, we can easily backport it to all supported versions.

B A
Screenshot From 2025-08-28 22-26-29 Screenshot From 2025-08-28 22-27-39

The contactsinteraction translations are not loaded in the contacts app
hence the source string is never translated.

Signed-off-by: Daniel Kesselberg <[email protected]>
@kesselb kesselb self-assigned this Aug 28, 2025
@kesselb kesselb added bug Something isn't working 3. to review Waiting for reviews labels Aug 28, 2025
@codecov
Copy link

codecov bot commented Aug 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 9.71%. Comparing base (02cb3fe) to head (890d97d).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##              main   #4662   +/-   ##
=======================================
  Coverage     9.71%   9.71%           
  Complexity     266     266           
=======================================
  Files          126     126           
  Lines         6330    6330           
  Branches      1184    1184           
=======================================
  Hits           615     615           
  Misses        5594    5594           
  Partials       121     121           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kesselb kesselb merged commit 8aaa14a into main Aug 29, 2025
34 checks passed
@kesselb kesselb deleted the fix/translation-for-recently branch August 29, 2025 12:20
@kesselb
Copy link
Contributor Author

kesselb commented Aug 29, 2025

/backport to stable7.3

@kesselb
Copy link
Contributor Author

kesselb commented Aug 29, 2025

/backport to stable6.0

@kesselb
Copy link
Contributor Author

kesselb commented Aug 29, 2025

/backport to stable8.0

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

Labels

3. to review Waiting for reviews bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants