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

[FIX] don't use private fields, use the API #34

Merged
merged 2 commits into from
Apr 6, 2016

Conversation

hbrunn
Copy link
Member

@hbrunn hbrunn commented Feb 15, 2016

version 0.2 of the postcode api supports as well protocol version 1 as 2, and in version 2, the internal data structures are different. By sticking to what the API offers, the addons works in both cases.

@lfreeke
Copy link
Contributor

lfreeke commented Feb 16, 2016

👍 (test)

@hbrunn hbrunn modified the milestones: 8.0, 7.0 Mar 1, 2016
@hbrunn
Copy link
Member Author

hbrunn commented Apr 2, 2016

@OCA/local-netherlands-maintainers anyone?

@astirpe
Copy link
Member

astirpe commented Apr 2, 2016

👍

1 similar comment
@StefanRijnhart
Copy link
Member

👍

@StefanRijnhart StefanRijnhart merged commit a61a886 into OCA:7.0 Apr 6, 2016
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.

None yet

4 participants