Opened 16 years ago

Closed 12 years ago

#7994 closed New feature (invalid)

Dutch Bank Account Number Field

Reported by: Rudolph Froger Owned by: nobody
Component: contrib.localflavor Version: dev
Severity: Normal Keywords: localflavorsplit
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

I created a Dutch Bank Account Number Field to the Dutch localflavor and also included tests.

Attachments (4)

DutchBankAccountNumberField.diff (5.8 KB ) - added by Rudolph Froger 16 years ago.
Removed a typo and altered the comments.
DutchBankAccountNumberField.2.diff (5.8 KB ) - added by Rudolph Froger 16 years ago.
Removed a typo and altered the comments.
DutchBankAccountNumberField.3.diff (8.8 KB ) - added by Rudolph Froger 16 years ago.
Removed a typo and altered the comments (sorry, previous patch did not change anything)
DutchBankAccountNumberField.4.diff (2.9 KB ) - added by Rudolph Froger 16 years ago.
Finally the correct patch, sorry for the noise.

Download all attachments as: .zip

Change History (12)

by Rudolph Froger, 16 years ago

Removed a typo and altered the comments.

by Rudolph Froger, 16 years ago

Removed a typo and altered the comments.

by Rudolph Froger, 16 years ago

Removed a typo and altered the comments (sorry, previous patch did not change anything)

by Rudolph Froger, 16 years ago

Finally the correct patch, sorry for the noise.

comment:1 by Eric Holscher, 16 years ago

milestone: 1.0 maybe
Triage Stage: UnreviewedDesign decision needed

comment:2 by Jacob, 16 years ago

milestone: 1.0 maybepost-1.0
Triage Stage: Design decision neededAccepted

comment:3 by (none), 15 years ago

milestone: post-1.0

Milestone post-1.0 deleted

comment:4 by Rudolph Froger, 15 years ago

Latest patch still applies cleanly and all tests pass.

comment:5 by Luke Plant, 13 years ago

Severity: Normal
Type: New feature

comment:6 by Julien Phalip, 13 years ago

Easy pickings: unset
Patch needs improvement: set

The tests would need to be rewritten using unittests since this is now Django's preferred way.

comment:7 by Aymeric Augustin, 12 years ago

UI/UX: unset

Change UI/UX from NULL to False.

comment:8 by Aymeric Augustin, 12 years ago

Keywords: localflavorsplit added
Resolution: invalid
Status: newclosed

django.contrib.localflavor is now deprecated — see https://docs.djangoproject.com/en/dev/ref/contrib/localflavor/

A repository was created for each localflavor at https://github.com/django/django-localflavor-? (Replace with the country code.)

If you're still interested in this ticket, could you create a pull request on that repository?

Sorry for not resolving this issue earlier, and thanks for your input!

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