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

Map with existing latitude/longitude #46

Open
bashu opened this issue May 30, 2015 · 8 comments
Open

Map with existing latitude/longitude #46

bashu opened this issue May 30, 2015 · 8 comments
Assignees

Comments

@bashu
Copy link
Owner

bashu commented May 30, 2015

Have you considered adding the option to map based solely on the latitude/longitude?

The template tag would look something like

{% easy_map

| [ ] [] [using <template_name>] %}

The use case is for mapping points in the ocean so using reverse geocoding won't work.


@bashu
Copy link
Owner Author

bashu commented May 30, 2015

Yes, that would be a useful feature, pull requests are welcome :)

Support for GeoDjango's PointFields would also be nice (it should be quite easy to implement by the way).

At the moment as a workaround you can create easy_maps.models.Address instance manually with the desired coordinates.


Original comment by: Mikhail Korobov

@bashu
Copy link
Owner Author

bashu commented May 30, 2015

Created a pull request for this at https://bitbucket.org/kmike/django-easy-maps/pull-request/2/added-feature-to-allow-latitude-and

Let me know what you think.


Original comment by: everett_toews

@bashu
Copy link
Owner Author

bashu commented May 30, 2015

Thanks! Sorry for a long delay. I'll post the comments to the pull request.


Original comment by: Mikhail Korobov

@bashu
Copy link
Owner Author

bashu commented May 30, 2015

Mikhail, that would be great addition to the map, I'm also waiting on this feature ;) I'll try to use create an Address instance manually and insert the cords from GPS , see if ti works.

Thank you so much for your work on this!


Original comment by: Sivan Greenberg

@bashu
Copy link
Owner Author

bashu commented May 30, 2015

Hey Mikhail, have you merged the pull request? Not pushing, just finding out if to apply it myself to my local branch :)


Original comment by: Sivan Greenberg

@bashu
Copy link
Owner Author

bashu commented May 30, 2015

Hi Sivan,

No, unfortunately I didn't had time to properly review and merge this the pull request; help with this is appreciated.


Original comment by: Mikhail Korobov

@bashu
Copy link
Owner Author

bashu commented May 30, 2015

Oh, excellent. I just encountered easy-map, and I also have the coordinates already. You just saved me creating my own pull request.


Original comment by: Itay Zandbank

@bashu bashu self-assigned this Oct 4, 2015
@jinchuika
Copy link

Hi. This was never merged, wasn't it? I still trying to find the way to use raw coordinates

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