Skip to content
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

Remove incorrect zipcodes #45

Merged
merged 4 commits into from
Nov 16, 2024

Conversation

danielfahy
Copy link

@danielfahy danielfahy commented Nov 4, 2024

Thank you for providing this gem it's really beneficial for zip sense checks!

I've noticed 4 invalid zip codes are returning as if they are valid.

For convenience, here's proof taken today, of all of the removed zip codes from the USPS website.
If USPS is not considered the true authority on zip validity for this gems purposes, please feel free to close out this PR

Thank you

46983
Screenshot 2024-11-04 at 12 41 26 PM

27838
Screenshot 2024-11-04 at 12 41 56 PM

29077
Screenshot 2024-11-04 at 12 42 22 PM

75286
Screenshot 2024-11-04 at 12 49 36 PM

Copy link
Collaborator

@brodyhoskins brodyhoskins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great catch, I've confirmed the removals are valid. Thank you!

@brodyhoskins brodyhoskins merged commit 7ad9a8c into monterail:master Nov 16, 2024
@danielfahy
Copy link
Author

Happy to help! This gem saved us so much trouble!

I think there's one zip code in Texas that is valid (and is in the source code) but somehow seems to be being rejected as invalid

Will see if I can dig further and open another request in the next week or so

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants