Opened 7 years ago

Closed 7 years ago

Last modified 3 years ago

#8661 closed (fixed)

French translations for Django 1.0

Reported by: david Owned by: david
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 7 years ago.
First draft, review welcome
django.po.french.2.diff (113.8 KB) - added by david 7 years ago.
Missing strings, now compilation works, thanks Kiz
django.po.french.3.diff (113.9 KB) - added by david 7 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 7 years ago.
A couple of updates thanks to benoitc's review
django.po.french.5.diff (17.9 KB) - added by david 7 years ago.
Add the latest(?) string before 1.0
django.po.french.6.diff (25.5 KB) - added by david 7 years ago.
Remove fuzzy comments + last review and some fixes
django+js.po.french.7.diff (31.7 KB) - added by david 7 years ago.
Found untranslated string in js.po, fixed now
django.po.french.8.diff (16.5 KB) - added by david 7 years ago.
Update string modified by security fix

Download all attachments as: .zip

Change History (24)

comment:1 Changed 7 years ago by mtredinnick

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

First draft, review welcome

comment:2 Changed 7 years ago by david

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

Changed 7 years ago by david

Missing strings, now compilation works, thanks Kiz

comment:3 Changed 7 years ago by julien

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 7 years ago by julien

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 7 years ago by julien

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 7 years ago by Alex

  • Triage Stage changed from Unreviewed to Accepted

comment:7 Changed 7 years ago by david

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

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

comment:8 Changed 7 years ago by julien

I'm fine with your changes, thanks David.

comment:9 Changed 7 years ago by mtredinnick

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

Committed in [8710].

Changed 7 years ago by david

A couple of updates thanks to benoitc's review

comment:10 Changed 7 years ago by david

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:11 Changed 7 years ago by mtredinnick

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

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

Changed 7 years ago by david

Add the latest(?) string before 1.0

comment:12 Changed 7 years ago by david

  • Resolution fixed deleted
  • Status changed from closed to reopened

Latest string added.

Changed 7 years ago by david

Remove fuzzy comments + last review and some fixes

Changed 7 years ago by david

Found untranslated string in js.po, fixed now

comment:13 Changed 7 years ago by mtredinnick

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

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

Changed 7 years ago by david

Update string modified by security fix

comment:14 Changed 7 years ago by david

  • Resolution fixed deleted
  • Status changed from closed to reopened

Update string modified by security fix.

comment:15 Changed 7 years ago by mtredinnick

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

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

comment:16 Changed 3 years ago by jacob

  • milestone 1.0 deleted

Milestone 1.0 deleted

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