search

rhomeister / api_postcode_nl

MIT License
2 stars 3 forks source link

replace explicit mapping with dynamic mapping #4

Open koenpunt opened 8 years ago

koenpunt commented 8 years ago

this means that area will now be available as surface_area andstreet_name as street. So in terms of versioning this would be major change. Or 2 additional keys could be added to the hash for BC

rhomeister commented 8 years ago

@koenpunt I like what you've done here, but can you please make this backward compatible?

koenpunt commented 8 years ago

Currently on vacation, but will look into that when I'm back

Op 19 aug. 2016 om 22:49 heeft Ruben Stranders notifications@github.com het volgende geschreven:

@koenpunt I like what you've done here, but can you please make this backward compatible?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

rhomeister commented 8 years ago

Sure. Enjoy your vacation!

On Aug 20, 2016 19:26, "Koen Punt" notifications@github.com wrote:

Currently on vacation, but will look into that when I'm back

Op 19 aug. 2016 om 22:49 heeft Ruben Stranders notifications@github.com het volgende geschreven:

@koenpunt I like what you've done here, but can you please make this backward compatible?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/rhomeister/api_postcode_nl/pull/4#issuecomment-241231679, or mute the thread https://github.com/notifications/unsubscribe-auth/ABobTE28Qm81fhEXpp8MVOtIR2VU8fwkks5qh5sngaJpZM4ILEet .