Opened 8 years ago

Closed 8 years ago

#4108 closed (fixed)

[patch] french PO file

Reported by: Baptiste <baptiste.goupil@…> Owned by: hugo
Component: Translations Version: master
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

Updated localization (with ticket #4107).

Lot of stuff added because of localflavors. I dunno if we should translate them since they will be used only in their countries, so only in their languages... What do you think ?

Attachments (5)

diff.diff (57.5 KB) - added by Baptiste <baptiste.goupil@…> 8 years ago.
patch
diff.2.diff (58.3 KB) - added by Baptiste <baptiste.goupil@…> 8 years ago.
Updated patch with localflavours.
diff.3.diff (58.3 KB) - added by Baptiste <baptiste.goupil@…> 8 years ago.
diff.2.diff contained an error + localization of the string of contrib/auth/models.py (#4107)
diff.4.diff (58.9 KB) - added by Baptiste <baptiste.goupil@…> 8 years ago.
arg - I had forgotten some fuzzy strings. Will that be good one day ? :)
diff.5.diff (58.9 KB) - added by Baptiste <baptiste.goupil@…> 8 years ago.
Fixed the error, compile-messages now work

Download all attachments as: .zip

Change History (11)

Changed 8 years ago by Baptiste <baptiste.goupil@…>

patch

comment:1 Changed 8 years ago by mtredinnick

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

Localflavours will be used outside of the local language (in some cases there isn't just one local language, after all).

What if you are designing a site for French people travelling to Germany? You'll need a list of French states and I'll want to translate them into the terms commonly used in France for those regions. It's the same reason country and language names are marked for translation -- you aren't going to use Deutschland as the name of the country on a French language website.

comment:2 Changed 8 years ago by mtredinnick

Err .. replace "French states" with "German states" in the above. We are going for German states in the French language in that example.

I need more coffee. :-(

comment:3 Changed 8 years ago by Baptiste <baptiste.goupil@…>

Okay, fine. Damned, translating that is going to be awful :D

Changed 8 years ago by Baptiste <baptiste.goupil@…>

Updated patch with localflavours.

Changed 8 years ago by Baptiste <baptiste.goupil@…>

diff.2.diff contained an error + localization of the string of contrib/auth/models.py (#4107)

Changed 8 years ago by Baptiste <baptiste.goupil@…>

arg - I had forgotten some fuzzy strings. Will that be good one day ? :)

comment:4 Changed 8 years ago by mtredinnick

  • Component changed from Internationalization to Translations
  • Triage Stage changed from Ready for checkin to Accepted

I can't commit this yet: there is a format string error at line 1333. Please remember to run bin/compile-messages.py -l fr on the PO file before submitting to catch these. I guess it might be that you are accidently translating the newforms-admin branch instead of trunk -- we aren't applying translation patches to branches, so please only submit patches against trunk.

Move back to "ready for checkin" once the error has been fixed so that we know it's ready.

Changed 8 years ago by Baptiste <baptiste.goupil@…>

Fixed the error, compile-messages now work

comment:5 Changed 8 years ago by Baptiste <baptiste.goupil@…>

  • Triage Stage changed from Accepted to Ready for checkin

Sorry for the error, I'll now use compile-messages before uploading the patches ; I didn't know that tool.

comment:6 Changed 8 years ago by mtredinnick

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

(In [5070]) Fixed #4108 -- Updated French translation. Thanks, Baptiste.

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