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

Updated us/oh/miami to more up-to-date source #4640

Merged
merged 4 commits into from Oct 4, 2019

Conversation

@MiniCodeMonkey
Copy link
Contributor

commented Oct 4, 2019

This source has addresses from newer developments that were not present in the previous static zip file dump

This source has addresses from newer developments that were not present in the previous static zip file dump
@trescube

This comment has been minimized.

Copy link
Member

commented Oct 4, 2019

@openaddresses-bot

This comment has been minimized.

@openaddresses-bot

This comment has been minimized.

@MiniCodeMonkey

This comment has been minimized.

Copy link
Contributor Author

commented Oct 4, 2019

@trescube Thanks a bunch for pointing that out 🙌 Updated the source, and doing another sanity check when the source job has completed.

@MiniCodeMonkey

This comment has been minimized.

Copy link
Contributor Author

commented Oct 4, 2019

I can confirm that the data looks good AND that it has addresses from new developments in the county that were not in the previous dataset.

This is good to merge in my end.

@trescube

This comment has been minimized.

Copy link
Member

commented Oct 4, 2019

Thanks, @MiniCodeMonkey!

@trescube trescube merged commit 4cd635b into openaddresses:master Oct 4, 2019
2 checks passed
2 checks passed
ci/circleci Your tests passed on CircleCI!
Details
openaddresses/hooked Succeeded on sources/us/oh/miami.json
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.