Adding MXSocialSecurityNumberField to local flavor #42

Closed
wants to merge 8 commits into from

3 participants

@gerardo-orozco

Adding MXSocialSecurityNumberField to local flavor

@ramiro ramiro commented on an outdated diff May 4, 2012
@@ -563,6 +563,7 @@ answer newbie questions, and generally made Django that much better:
Gasper Zejn <zejn@kiberpipa.org>
Jarek Zgoda <jarek.zgoda@gmail.com>
Cheng Zhang
+ Francisco Albarran Cristobal <pahko.xd@gmail.com>
@ramiro
Django member
ramiro added a note May 4, 2012

Contributor names in the AUTHORS file need to be sorted by last name.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@ramiro ramiro commented on an outdated diff May 4, 2012
django/contrib/localflavor/mx/forms.py
+ following the next pattern:
+
+ ===== =======================================
+ Index Required numbers
+ ===== =======================================
+ 1-2 The number of the branch office where the Social Security Number
+ was designated.
+ 3-4 The year of inscription to the Social Security.
+ 5-6 The year of birth of the Social Security Number owner.
+ 7-10 The progressive number of procedure for the IMSS.
+ (This digit is provided exclusively by the Institute as it regards
+ the Folio number of such procedure).
+ 11 The verification digit.
+
+ More info about this:
+ <reference here>
@ramiro
Django member
ramiro added a note May 4, 2012

Please add the reference or simply remove the section.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@pahko

Now Fixed @ramiro thanks.

@adrianholovaty
Django member

Hey there -- django.contrib.localflavor is now deprecated, and we're not making any more changes to it (see https://docs.djangoproject.com/en/dev/ref/contrib/localflavor/). Could you reopen this pull request for the shiny new package django-localflavor-mx? Here's the link: https://github.com/django/django-localflavor-mx

Sorry we didn't get to this pull request before the deprecation. I hope it's not too much of a pain to migrate this to the new package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment