Skip to content

update french translation #2042

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

Merged
merged 4 commits into from
Feb 25, 2025

Conversation

merwok
Copy link
Contributor

@merwok merwok commented Feb 24, 2025

Problem

Untranslated strings in the FR localization

Solution

I translated most of them.

I also took the opportunity to add trimmed to a blocktranslation tag, to remove spaces and newlines from the translation entries.

And I removed the line numbers in the translation files: the filenames are very useful to look at the context for a translation entry, but the line numbers are not very useful and change all the time when the source files change, causing unnecessary churn and possible merge conflicts.

Acceptance Criteria

None

@matthewhegarty
Copy link
Contributor

matthewhegarty commented Feb 24, 2025

Thanks, looks like a sensible change.

Please could you add the --add-location=file flag to the makemessages command?

Feel free to update the Changelog and add your name to AUTHORS.

@coveralls
Copy link

coveralls commented Feb 24, 2025

Coverage Status

coverage: 100.0%. remained the same
when pulling a2ed5c8 on caravancoop:update-fr
into 1d2b9fe on django-import-export:main.

@merwok
Copy link
Contributor Author

merwok commented Feb 24, 2025

done!

@matthewhegarty matthewhegarty merged commit b8f0ad8 into django-import-export:main Feb 25, 2025
8 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.

None yet

3 participants