Opened 8 years ago

Closed 8 years ago

Last modified 5 years ago

#10013 closed (fixed)

Updated French translation

Reported by: Stephane Raimbault Owned by: Stephane Raimbault
Component: Translations Version: master
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: UI/UX:

Description

Claude Paroz and me have fixed the current French translation (many changes) and our changes have been reviewed by David Larlet.

Attachments (5)

django.po (107.7 KB) - added by Stephane Raimbault 8 years ago.
djangojs.po (3.1 KB) - added by Stephane Raimbault 8 years ago.
I've removed authors from the file and restored previous header with Mikaël Barbero
django-AUTHORS.patch (387 bytes) - added by Stephane Raimbault 8 years ago.
django-fr-po.diff (70.8 KB) - added by Stephane Raimbault 8 years ago.
Patch AUTHORS, French translation of django.po and djangojs.po
django-1.0.X-fr.patch (70.8 KB) - added by Stephane Raimbault 8 years ago.
AUTHORS (ordered), French translation of django.po and djangojs.po for the 1.0.X branch

Download all attachments as: .zip

Change History (18)

comment:1 Changed 8 years ago by David Larlet

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedReady for checkin

I confirm, really good work from Stéphane and Claude!

comment:2 Changed 8 years ago by Claude Paroz

Cc: Claude Paroz added

comment:3 Changed 8 years ago by (none)

milestone: post-1.0

Milestone post-1.0 deleted

comment:4 Changed 8 years ago by Malcolm Tredinnick

Patch needs improvement: set
Triage Stage: Ready for checkinAccepted

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 Changed 8 years ago by Stephane Raimbault

Owner: changed from nobody to Stephane Raimbault

Changed 8 years ago by Stephane Raimbault

Attachment: django.po added

Changed 8 years ago by Stephane Raimbault

Attachment: djangojs.po added

I've removed authors from the file and restored previous header with Mikaël Barbero

Changed 8 years ago by Stephane Raimbault

Attachment: django-AUTHORS.patch added

Changed 8 years ago by Stephane Raimbault

Attachment: django-fr-po.diff added

Patch AUTHORS, French translation of django.po and djangojs.po

comment:6 Changed 8 years ago by Stephane Raimbault

milestone: 1.1 beta
Patch needs improvement: unset
Triage Stage: AcceptedReady for checkin

Ready now!

comment:7 Changed 8 years ago by Stephane Raimbault

  • 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 Changed 8 years ago by David Larlet

Great work! Just a note on AUTHORS, they used to be ordered by last name :)

comment:9 Changed 8 years ago by Malcolm Tredinnick

Resolution: fixed
Status: newclosed

(In [9941]) Fixed #10013 -- Updated French translation.

Thanks, Claude Paroz and Stéphane Raimbault

comment:10 Changed 8 years ago by Malcolm Tredinnick

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).

Changed 8 years ago by Stephane Raimbault

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 Changed 8 years ago by Stephane Raimbault

milestone: 1.1 beta1.0.3
Resolution: fixed
Status: closedreopened

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 Changed 8 years ago by Malcolm Tredinnick

Resolution: fixed
Status: reopenedclosed

(In [9957]) [1.0.X] Fixed #10013 -- Updated French translation for 1.0 branch.

Thanks, Stéphane Raimbault and Claude Paroz.

comment:13 Changed 5 years ago by Jacob

milestone: 1.0.3

Milestone 1.0.3 deleted

Note: See TracTickets for help on using tickets.
Back to Top