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

npm update needed? #1

Closed
anetz89 opened this issue Jan 28, 2017 · 2 comments
Closed

npm update needed? #1

anetz89 opened this issue Jan 28, 2017 · 2 comments

Comments

@anetz89
Copy link

anetz89 commented Jan 28, 2017

Hi there,
great job so far! I wonder why your package is not popular...
I installed your npm package and it seems like the latest version of your repository is not available.

The following lines (in OverpassFrontent.js) breaks my code:

function toQuadtreeLookupBox (boundingbox) {
  return new Quadtree.Box(
    new Quadtree.Point(boundingbox.bounds.minlat, boundingbox.bounds.minlon),
    new Quadtree.Point(boundingbox.bounds.maxlat, boundingbox.bounds.maxlon)
  )
}

as boundingbox.bounds is not defined.
Your repository seems to fix this problem already (removing ".bounds" and access min/maxlon/lat direclty).

Can you update the package? Thanks in advance!

@plepe
Copy link
Owner

plepe commented Feb 2, 2017

The package is rather new, I developed it for my ptmap project. I'll try to build a new npm package soon, first I'll check that all tests are working ...

@anetz89
Copy link
Author

anetz89 commented Feb 7, 2017

No worries. To be honest, I switched to query-overpass, as it is more lightweight. Maybe it is helpful for your project, too.
Thanks!

@plepe plepe closed this as completed Aug 16, 2018
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