Skip to content

Translate fr #4168

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Translate fr #4168

wants to merge 3 commits into from

Conversation

miniupnp
Copy link
Contributor

Description of Changes

Add french translation for

  • editTableOfContents.import/export
  • pdfToImage.dpi
  • update.*

Checklist

General

Documentation

UI Changes (if applicable)

  • Screenshots or videos demonstrating the UI changes are attached (e.g., as comments or direct attachments in the PR)

Testing (if applicable)

  • I have tested my changes locally. Refer to the Testing Guide for more details.

note:
  DPI translate to PPP (points par pouce) but DPI is used elsewhere in the
  french translation.
@dosubot dosubot bot added the size:M This PR changes 30-99 lines ignoring generated files. label Aug 10, 2025
@stirlingbot
Copy link
Contributor

stirlingbot bot commented Aug 10, 2025

🚀 Translation Verification Summary

🔄 Reference Branch: main-branch-messages_en_GB.properties

📃 File Check: messages_fr_FR.properties

  1. Test Status:Failed
  • Issue:
    • Too many lines: 1904 (reference) vs 1905 (current). Please verify if there is an additional line that needs to be removed.
  1. Test Status:Passed
  2. Test Status:Passed

❌ Overall Check Status: Failed

@miniupnp please check your translation if it conforms to the standard. Follow the format of messages_en_GB.properties

@miniupnp
Copy link
Contributor Author

Too many lines: 1904 (reference) vs 1905 (current). Please verify if there is an additional line that needs to be removed.

I thought there was a missing blank line between the update.* and settings.*, that's why I added a blank line

@Ludy87 Ludy87 self-requested a review August 11, 2025 04:21
update.breakingChanges=Changements critiques:
update.breakingChangesDefault=Cette version contient des changements critiques
update.migrationGuide=Guide de migration

Copy link
Contributor

Choose a reason for hiding this comment

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

The review was very meticulous and therefore identified the line as an "error," but it's not a problem. It will be corrected later in the audit.

Copy link
Contributor

Choose a reason for hiding this comment

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

@Frooodle please merge, its ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:M This PR changes 30-99 lines ignoring generated files. Translation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants