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

Searches near 'Me' and 'marker' are not working #257

Closed
rladines opened this issue Nov 4, 2016 · 6 comments
Closed

Searches near 'Me' and 'marker' are not working #257

rladines opened this issue Nov 4, 2016 · 6 comments
Assignees
Labels
Milestone

Comments

@rladines
Copy link
Collaborator

rladines commented Nov 4, 2016

report by @reve0716: searches using Location of Near "Me" and "marker" are getting an error "Cannot read property 'layerPointToLatLng' of undefined"

[priya]: this is happening in WG>INT environment

@rladines rladines added the bug label Nov 4, 2016
@rladines rladines added this to the v1.3.0 milestone Nov 4, 2016
@rladines rladines self-assigned this Nov 4, 2016
@rladines
Copy link
Collaborator Author

rladines commented Nov 4, 2016

After researching on this, it was found that this is due to a reported issue with Leaflet 1.0, Leaflet/Leaflet#4978; will look into an alternative solution to get the geometry of the marker bounds

@PriyaDoIT
Copy link

related to #255

@rladines
Copy link
Collaborator Author

rladines commented Nov 8, 2016

Was able to workaround the Leaflet issue by overriding the L.Circle behavior with 0.7.3's which was proven to work with our code. Will apply any future Leaflet patch later once this is fixed in their repo.

Read for re-test.

@rladines rladines assigned reve0716 and PriyaDoIT and unassigned rladines Nov 8, 2016
@PriyaDoIT
Copy link

Tested this issue on INT using both Marker and Around Me. Passed.

@reve0716
Copy link
Contributor

reve0716 commented Nov 9, 2016

I completed the testing for markers and near me - Its a Go

@rladines
Copy link
Collaborator Author

Closing based on comments from @PriyaDoIT and @reve0716

@rladines rladines removed the Alpha label Nov 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants