#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 , 17 years ago
| Triage Stage: | Unreviewed → Ready for checkin |
|---|
comment:2 by , 17 years ago
| Cc: | added |
|---|
comment:4 by , 17 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 , 17 years ago
| Owner: | changed from to |
|---|
by , 17 years ago
by , 17 years ago
| Attachment: | djangojs.po added |
|---|
I've removed authors from the file and restored previous header with Mikaël Barbero
by , 17 years ago
| Attachment: | django-AUTHORS.patch added |
|---|
by , 17 years ago
| Attachment: | django-fr-po.diff added |
|---|
Patch AUTHORS, French translation of django.po and djangojs.po
comment:6 by , 17 years ago
| milestone: | → 1.1 beta |
|---|---|
| Patch needs improvement: | unset |
| Triage Stage: | Accepted → Ready for checkin |
Ready now!
comment:7 by , 17 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 , 17 years ago
Great work! Just a note on AUTHORS, they used to be ordered by last name :)
comment:9 by , 17 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:10 by , 17 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 , 17 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 , 17 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 , 17 years ago
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
I confirm, really good work from Stéphane and Claude!