-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add PIN Code range and union territories in en_IN
address provider
#2091
base: master
Are you sure you want to change the base?
Conversation
en_IN - add pin code range in address
"Jammu and Kashmir", | ||
"Ladakh", | ||
"Lakshadweep", | ||
("Pondicherry, Puducherry"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it fine being a tuple?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would make them all tuples, even if they just have one element
"Jammu and Kashmir", | ||
"Ladakh", | ||
"Lakshadweep", | ||
("Pondicherry, Puducherry"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would make them all tuples, even if they just have one element
"HP": (170_000, 179_999), | ||
"JH": (800_000, 859_999), | ||
"KA": (560_000, 599_999), | ||
"KL": [(670_000, 681_999), (683_000, 699_999)], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we make all items lists of tuples?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I make them list of tuples, I don't see how it would make a difference since it's meant to be never changed (unless a new state is created). I don't mind changing it to list of tuples as well. I'm just asking for your view
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather have all valueswith the same structure
What does this change
Updates the
address
provider foren_IN
to add union territories and PIN code ranges for statesWhat was wrong
No state / union territories pin code specified
How this fixes it
Add state / union territories pin codes with ranges
Fixes #2089