Opened 18 years ago

Closed 17 years ago

#3986 closed (fixed)

Add Indian Localflavour

Reported by: pradeep.gowda@… Owned by: Adrian Holovaty
Component: contrib.localflavor Version: dev
Severity: Keywords: localflavor, l10n, in, ind
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Patch to add a module to django.contrib.localflavor for India: INStateSelect and INZipCodeField.

note: The directory name is ind instead of two letter in as it conflicts with python keyword in.

Attachments (2)

indianlocalflavour.diff​ (4.3 KB ) - added by pradeep.gowda@… 18 years ago.
initial diff file attached.
in_localflavor.diff​ (4.3 KB ) - added by Simon G. <dev@…> 17 years ago.
renamed ind/ to in_/ in the original patch to comply with PEP8

Download all attachments as: .zip

Change History (12)

by pradeep.gowda@…, 18 years ago

Attachment: indianlocalflavour.diff​ added

initial diff file attached.

comment:1 by Simon G. <dev@…>, 18 years ago

Triage Stage: Unreviewed → Ready for checkin

comment:2 by Russell Keith-Magee, 18 years ago

Needs tests: set
Patch needs improvement: set

To comply with PEP8, the name of this module should be 'in_', not 'ind'.

comment:3 by Russell Keith-Magee, 18 years ago

Triage Stage: Ready for checkin → Accepted

by Simon G. <dev@…>, 17 years ago

Attachment: in_localflavor.diff​ added

renamed ind/ to in_/ in the original patch to comply with PEP8

comment:4 by Simon G. <dev@…>, 17 years ago

Patch needs improvement: unset
Triage Stage: Accepted → Ready for checkin

comment:5 by Malcolm Tredinnick, 17 years ago

Resolution: → fixed
Status: new → closed

(In [5847]) Fixed #3986 -- Added Indian localflavor. Thanks, pradeep.gowda@….

comment:6 by C8E, 17 years ago

Resolution: fixed
Status: closed → reopened

Malcolm, you did apply the patch in django/contrib/in_ , not in django/contrib/localflavor/in_ . Is that intentional?

comment:7 by Malcolm Tredinnick, 17 years ago

Nope, not intentional. I screwed up. :-(

Good catch. I'll fix it.

comment:8 by Malcolm Tredinnick, 17 years ago

Resolution: → fixed
Status: reopened → closed

(In [5856]) Moved in_ directory to its correct location under localflavor. I committed it in the wrong place in [5847]. Fixed #3986.

comment:9 by Simon G. <dev@…>, 17 years ago

Resolution: fixed
Status: closed → reopened

My fault sorry - should have updated the patch to the right dir. Malcolm - the in_ dir. is still in django/contrib/ but empty. Can this be removed?

comment:10 by Simon G. <dev@…>, 17 years ago

Resolution: → fixed
Status: reopened → closed

Already well ahead of me - fixed in [5879]

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