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

Integration with plots2 . #40

Open
sagarpreet-chadha opened this issue Feb 19, 2019 · 2 comments
Open

Integration with plots2 . #40

sagarpreet-chadha opened this issue Feb 19, 2019 · 2 comments

Comments

@sagarpreet-chadha
Copy link
Collaborator

We only need to integrate it to /people for now .

The current LBLD API definition looks fine , we just have to pass BlurredLocation object AND integration is done 😄 !

What more changes should be done to make LBLD API more easy to integrate ?

@jywarren
Copy link
Member

Noting in #39 (comment) that we might default to geoJSON but show an example response handler written for the plots2 API.

Also noting we may want to have an optional "precision" parameter and think how this would be used to add an extra layer of protection to be sure we truncate properly?

@sagarpreet-chadha
Copy link
Collaborator Author

Hi @jywarren , lets release this library to npm . What do you think ? Thanks :)

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

2 participants