Skip to content

[IMP] support the v2 api#10

Merged
steffex merged 2 commits intosteffex:masterfrom
hbrunn:master-v2
Dec 8, 2015
Merged

[IMP] support the v2 api#10
steffex merged 2 commits intosteffex:masterfrom
hbrunn:master-v2

Conversation

@hbrunn
Copy link
Contributor

@hbrunn hbrunn commented Dec 3, 2015

This is a preparation to support the v2 api (http://www.postcodeapi.nu/docs/)

For now, the default is v1, but we can change this to v2 when v1 will be deactivated in March

@hbrunn
Copy link
Contributor Author

hbrunn commented Dec 3, 2015

@StefanRijnhart @NL66278 might be interested in this

@steffex
Copy link
Owner

steffex commented Dec 7, 2015

Good job @hbrunn. Had it on my todo list for the end of the month :)

Perhaps it's a good idea to also implement the unique address ID in the result, in case you want to compare the result address to a previous fetched one.

@steffex steffex added this to the v0.2 milestone Dec 7, 2015
steffex added a commit that referenced this pull request Dec 8, 2015
@steffex steffex merged commit 595d753 into steffex:master Dec 8, 2015
@hbrunn
Copy link
Contributor Author

hbrunn commented Dec 8, 2015

thanks for the merge!

@StefanRijnhart
Copy link
Collaborator

Excellent, @hbunn! Is there a reason to default to v1? Defaulting to v2 now saves everyone another software update before march.

@hbrunn
Copy link
Contributor Author

hbrunn commented Dec 11, 2015

@StefanRijnhart you need a new api key for v2, so I figured it might be more convenient to have users not needing to change anything right now, but only when v1 is dead for good

@StefanRijnhart
Copy link
Collaborator

Ah, I see.

@StefanRijnhart
Copy link
Collaborator

@steffex care to attempt a release on Pypi?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants