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

Add param for area type to /postcode and /point #270

Open
BenJam opened this issue Nov 7, 2016 · 3 comments
Open

Add param for area type to /postcode and /point #270

BenJam opened this issue Nov 7, 2016 · 3 comments

Comments

@BenJam
Copy link

BenJam commented Nov 7, 2016

in order to simplify the processing and reduce i/o for clients of the API

@dracos
Copy link
Member

dracos commented Nov 7, 2016

/point already has this? :) http://mapit.mysociety.org/point/27700/400000,300000.html?type=MTD

@dracos
Copy link
Member

dracos commented Nov 7, 2016

Can add for /postcode to reduce i/o a bit as you say, but I don't think it will simplify any processing that I can think of.

@torotil
Copy link
Contributor

torotil commented May 23, 2019

[…] I don't think it will simplify any processing that I can think of.

Usually downstream applications that use the API are only interested in some of the area types. So usually they would:

  1. Get all areas for a postcode.
  2. Filter the areas by type.

… before doing any further work on them.

Adding a type parameter would make 2. unnecessary. I’d vote for it to allow for a list of types though (ie. ?type=CCG,SHB,…).

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

No branches or pull requests

3 participants