Skip to content

Fr translation#4414

Merged
hoangdat merged 5 commits intomaintenance_v0.26.xfrom
fr_translation
Mar 26, 2026
Merged

Fr translation#4414
hoangdat merged 5 commits intomaintenance_v0.26.xfrom
fr_translation

Conversation

@hoangdat
Copy link
Copy Markdown
Member

@hoangdat hoangdat commented Mar 26, 2026

Summary by CodeRabbit

Release Notes

  • Bug Fixes

    • Corrected French read-receipt terminology for improved accuracy
  • Documentation

    • Refined French mailbox and thread toggle interface labels for clarity
    • Added comprehensive localization strings for label and tag management across French, Russian, and Vietnamese languages

Facyla and others added 3 commits March 26, 2026 09:27
Currently translated at 100.0% (812 of 812 strings)

Translation: Linagora/TeamMail
Translate-URL: https://hosted.weblate.org/projects/linagora/teammail/fr/
Currently translated at 98.8% (848 of 858 strings)

Translation: Linagora/TeamMail
Translate-URL: https://hosted.weblate.org/projects/linagora/teammail/fr/
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e8b4d07a-6bff-4325-85df-2198d9e8f68c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

This pull request updates localization files across three languages. The French localization file (intl_fr.arb) replaces "accusé de réception" with "accusé de lecture" in multiple read-receipt strings, corrects thread toggle terminology, and refines mailbox list labeling. Additionally, new label/tag management localization entries are added to French, Russian (intl_ru.arb), and Vietnamese (intl_vi.arb) files. These new entries cover label creation, color selection, error handling, visibility settings, renaming, deletion confirmation, and optional descriptions across all three language variants.

Suggested reviewers

  • tddang-linagora
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Fr translation' is too vague and generic; it does not convey specific information about the actual changes made in the pull request. Replace with a more specific title such as 'Update French translations for read receipts and add label management strings' to clearly communicate the main changes.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fr_translation

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Member

@dab246 dab246 left a comment

Choose a reason for hiding this comment

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

lgtm

@hoangdat
Copy link
Copy Markdown
Member Author

@CodeRabbit please review

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

@hoangdat: Sure, I'll review the French translation changes in this PR right away!

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link
Copy Markdown
Member

@dab246 dab246 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@lib/l10n/intl_ru.arb`:
- Around line 5755-5823: Add the 11 missing label-management keys to the English
base ARB (intl_en.arb) so gen-l10n generates accessors:
pleaseEnterNameYourNewLabel, chooseALabelColor, tagNameAlreadyExists,
chooseCustomColour, createNewLabelFailure, labelVisibilitySettingExplanation,
labelVisibilityToggleDescription, updateTheLabelName,
areYouSureYouWantToDeleteTheLabel, labelDescription, and
labelDescriptionHintText; supply appropriate English strings for each key and
include the matching metadata entries (the `@-entries`), ensuring the
areYouSureYouWantToDeleteTheLabel metadata contains the placeholder "labelName"
(placeholders_order and placeholders) to mirror the Russian file.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 6258861d-905e-4e18-86c7-223d94276348

📥 Commits

Reviewing files that changed from the base of the PR and between 16b1de1 and 7ec1dfe.

📒 Files selected for processing (3)
  • lib/l10n/intl_fr.arb
  • lib/l10n/intl_ru.arb
  • lib/l10n/intl_vi.arb

@hoangdat hoangdat merged commit b30a4d5 into maintenance_v0.26.x Mar 26, 2026
19 checks passed
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.

3 participants