Code

Opened 5 years ago

Closed 5 years ago

Last modified 3 years ago

#10013 closed (fixed)

Updated French translation

Reported by: stephaner Owned by: stephaner
Component: Translations Version: master
Severity: Keywords:
Cc: claudep 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 stephaner 5 years ago.
djangojs.po (3.1 KB) - added by stephaner 5 years ago.
I've removed authors from the file and restored previous header with Mikaël Barbero
django-AUTHORS.patch (387 bytes) - added by stephaner 5 years ago.
django-fr-po.diff (70.8 KB) - added by stephaner 5 years ago.
Patch AUTHORS, French translation of django.po and djangojs.po
django-1.0.X-fr.patch (70.8 KB) - added by stephaner 5 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 5 years ago by david

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Ready for checkin

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

comment:2 Changed 5 years ago by claudep

  • Cc claudep added

comment:3 Changed 5 years ago by anonymous

  • milestone post-1.0 deleted

Milestone post-1.0 deleted

comment:4 Changed 5 years ago by mtredinnick

  • Patch needs improvement set
  • Triage Stage changed from Ready for checkin to 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 Changed 5 years ago by stephaner

  • Owner changed from nobody to stephaner

Changed 5 years ago by stephaner

Changed 5 years ago by stephaner

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

Changed 5 years ago by stephaner

Changed 5 years ago by stephaner

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

comment:6 Changed 5 years ago by stephaner

  • milestone set to 1.1 beta
  • Patch needs improvement unset
  • Triage Stage changed from Accepted to Ready for checkin

Ready now!

comment:7 Changed 5 years ago by stephaner

  • 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 5 years ago by david

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

comment:9 Changed 5 years ago by mtredinnick

  • Resolution set to fixed
  • Status changed from new to closed

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

Thanks, Claude Paroz and Stéphane Raimbault

comment:10 Changed 5 years ago by mtredinnick

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 5 years ago by stephaner

AUTHORS (ordered), French translation of django.po and djangojs.po for the 1.0.X branch

comment:11 Changed 5 years ago by stephaner

  • milestone changed from 1.1 beta to 1.0.3
  • Resolution fixed deleted
  • Status changed from closed to 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 Changed 5 years ago by mtredinnick

  • Resolution set to fixed
  • Status changed from reopened to closed

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

Thanks, Stéphane Raimbault and Claude Paroz.

comment:13 Changed 3 years ago by jacob

  • milestone 1.0.3 deleted

Milestone 1.0.3 deleted

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.