#10013 closed (fixed)
Updated French translation
Reported by: | Stephane Raimbault | Owned by: | Stephane Raimbault |
---|---|---|---|
Component: | Translations | Version: | dev |
Severity: | Keywords: | ||
Cc: | Claude Paroz | Triage Stage: | Ready for checkin |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Claude Paroz and me have fixed the current French translation (many changes) and our changes have been reviewed by David Larlet.
Attachments (5)
Change History (18)
comment:1 by , 16 years ago
Triage Stage: | Unreviewed → Ready for checkin |
---|
comment:2 by , 16 years ago
Cc: | added |
---|
comment:4 by , 16 years ago
Patch needs improvement: | set |
---|---|
Triage Stage: | Ready for checkin → Accepted |
This patch introduces errors into the PO file. It is always necessary to check the format of your changes by running
django-admin.py compilemessages -l fr
from the django/
directory. If you do that, you'll notice a bunch of problems are reported. In particular, you cannot have both msgstr[0]
and msgstr
lines for a single msgid
.
Also, we do not put individual contributors' names into any file in Django (except for the last-translator name). So the names you have put into the comments at the top of the file should actually be patches against the AUTHORS file. David's name is already in AUTHORS. In the revised patch, please put the other two names into AUTHORS.
Finally, as proposed in the first comment to #10133, please include the fix mentioned there, so that there's only a single patch to apply when the time comes. It's always a little fragile applying two patches to the same locale files.
comment:5 by , 16 years ago
Owner: | changed from | to
---|
by , 16 years ago
by , 16 years ago
Attachment: | djangojs.po added |
---|
I've removed authors from the file and restored previous header with Mikaël Barbero
by , 16 years ago
Attachment: | django-AUTHORS.patch added |
---|
by , 16 years ago
Attachment: | django-fr-po.diff added |
---|
Patch AUTHORS, French translation of django.po and djangojs.po
comment:6 by , 16 years ago
milestone: | → 1.1 beta |
---|---|
Patch needs improvement: | unset |
Triage Stage: | Accepted → Ready for checkin |
Ready now!
comment:7 by , 16 years ago
- Fixed plural forms
- Add the Plural-forms description in the PO header
- Merged with #10133
- Updated translation (after makemessages)
- Moved author names in AUTHORS
comment:8 by , 16 years ago
Great work! Just a note on AUTHORS, they used to be ordered by last name :)
comment:9 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:10 by , 16 years ago
The patch doesn't apply cleanly (or even after a little hitting with hammers) to the 1.0.X branch. I've said in the past that I won't backport anything that doesn't apply cleanly there. So if you guys also want to update the 1.0.X version, you'll need to create a separate patch.
You'll also see that I've fixed the AUTHORS portion of this patch when I applied it (as David noted -- names are ordered by surname).
by , 16 years ago
Attachment: | django-1.0.X-fr.patch added |
---|
AUTHORS (ordered), French translation of django.po and djangojs.po for the 1.0.X branch
comment:11 by , 16 years ago
milestone: | 1.1 beta → 1.0.3 |
---|---|
Resolution: | fixed |
Status: | closed → reopened |
I don't intend to patch the 1.0.X branch, but why not!
So I've added an clean patch for that branch (/branches/releases/1.0.X) and changed the ticket milestone accordingly.
comment:12 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
I confirm, really good work from Stéphane and Claude!