Opened 8 years ago

Closed 8 years ago

Last modified 5 years ago

#8661 closed (fixed)

French translations for Django 1.0

Reported by: David Larlet Owned by: David Larlet
Component: Translations Version: master
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

I'll fix #8481 in the meantime.

Attachments (8)

django.po.french.diff (113.7 KB) - added by David Larlet 8 years ago.
First draft, review welcome
django.po.french.2.diff (113.8 KB) - added by David Larlet 8 years ago.
Missing strings, now compilation works, thanks Kiz
django.po.french.3.diff (113.9 KB) - added by David Larlet 8 years ago.
Better translations thanks to julien's review (comment:4 and comment:5)
django.po.french.4.diff (57.4 KB) - added by David Larlet 8 years ago.
A couple of updates thanks to benoitc's review
django.po.french.5.diff (17.9 KB) - added by David Larlet 8 years ago.
Add the latest(?) string before 1.0
django.po.french.6.diff (25.5 KB) - added by David Larlet 8 years ago.
Remove fuzzy comments + last review and some fixes
django+js.po.french.7.diff (31.7 KB) - added by David Larlet 8 years ago.
Found untranslated string in js.po, fixed now
django.po.french.8.diff (16.5 KB) - added by David Larlet 8 years ago.
Update string modified by security fix

Download all attachments as: .zip

Change History (24)

comment:1 Changed 8 years ago by Malcolm Tredinnick

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset

What does this mean? Is it a dupe of #8481? Or meant to replace it? This ticket doesn't seem to have any valid resolution.

Changed 8 years ago by David Larlet

Attachment: django.po.french.diff added

First draft, review welcome

comment:2 Changed 8 years ago by David Larlet

@mtredinnick: it's just a ticket to attach French translations (and I had included #8481 patch so I'll close it now).

Changed 8 years ago by David Larlet

Attachment: django.po.french.2.diff added

Missing strings, now compilation works, thanks Kiz

comment:3 Changed 8 years ago by Julien Phalip

David, I'm reviewing the patch. It looks great but I found a few typos. Will send a report very soon so please wait a bit before committing this, it takes a while to read through ;)

comment:4 Changed 8 years ago by Julien Phalip

David, I reviewed the patch and it looks good. There are just small typos/mistakes. I'll just enumerate them below as I'm not too familiar with the translation system (.po and all) and I don't want to make a mess with the patch ;)

  • Vous pouvez continuez de l'éditez ci-dessous. should be: ...continuer de l'éditer ci-dessous
  • "L'objet %(name)s \"%(obj)s\" a été ajouté avec succès. Vous pouvez continuez de l'éditez ci-dessous." should be: ...continuer de l'éditer ci-dessous.
  • Nous vous avons envoyé par courriel les instructions pour changer de mot de passe à l'adresse que vous avez spécifié. Vous devriez le recevoir rapidement. should be: ...spécifiée...
  • This comment was posted by an authenticated user and thus the name is read-only. should be: authentifié , not non authentifié . This appears twice.

I hope my comments are not too hard to read ;)

comment:5 Changed 8 years ago by Julien Phalip

Here are a couple more remarks:

  • Entrez une date et une heure valide au format AAAA-MM-JJ HH:MM[:ss[.uuuuuu]]. should be: ...valides...
  • user's URL should not just be: utilisateurs . "URL" is missing here.

comment:6 Changed 8 years ago by Alex Gaynor

Triage Stage: UnreviewedAccepted

comment:7 Changed 8 years ago by David Larlet

Thanks for your review julien! Here are my thoughts:

  • continuer de l'éditer -> continuer l'édition (make sense?)
  • spécifiée: OK
  • authentifié: ouch...
  • valides: OK
  • user's URL -> URL de l'utilisateur: same for courriel (I'm not sure where it is used, someone can confirm?)

I know that's a pain to review a .po patch but I can't find a better way to do this.

Changed 8 years ago by David Larlet

Attachment: django.po.french.3.diff added

Better translations thanks to julien's review (comment:4 and comment:5)

comment:8 Changed 8 years ago by Julien Phalip

I'm fine with your changes, thanks David.

comment:9 Changed 8 years ago by Malcolm Tredinnick

Resolution: fixed
Status: newclosed

Committed in [8710].

Changed 8 years ago by David Larlet

Attachment: django.po.french.4.diff added

A couple of updates thanks to benoitc's review

comment:10 Changed 8 years ago by David Larlet

Resolution: fixed
Status: closedreopened

comment:11 Changed 8 years ago by Malcolm Tredinnick

Resolution: fixed
Status: reopenedclosed

(In [8725]) Another French update from David Larlet. Fixed #8661.

Changed 8 years ago by David Larlet

Attachment: django.po.french.5.diff added

Add the latest(?) string before 1.0

comment:12 Changed 8 years ago by David Larlet

Resolution: fixed
Status: closedreopened

Latest string added.

Changed 8 years ago by David Larlet

Attachment: django.po.french.6.diff added

Remove fuzzy comments + last review and some fixes

Changed 8 years ago by David Larlet

Attachment: django+js.po.french.7.diff added

Found untranslated string in js.po, fixed now

comment:13 Changed 8 years ago by Malcolm Tredinnick

Resolution: fixed
Status: reopenedclosed

(In [8834]) French translation update from David Larlet. Fixed #8661.

Changed 8 years ago by David Larlet

Attachment: django.po.french.8.diff added

Update string modified by security fix

comment:14 Changed 8 years ago by David Larlet

Resolution: fixed
Status: closedreopened

Update string modified by security fix.

comment:15 Changed 8 years ago by Malcolm Tredinnick

Resolution: fixed
Status: reopenedclosed

(In [8895]) Fixed #8661 -- Updated French translation from David Larlet.

comment:16 Changed 5 years ago by Jacob

milestone: 1.0

Milestone 1.0 deleted

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