#26697 closed Cleanup/optimization (fixed)
Drop django.contrib.gis.maps?
| Reported by: | Ramiro Morales | Owned by: | nobody |
|---|---|---|---|
| Component: | GIS | Version: | dev |
| Severity: | Normal | Keywords: | maps gmaps gis |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
- The django.contrib.gis.maps.openlayers module has been empty since it was created back in 2007.
- The django.contrib.gis.maps.google module:
- Interfaces with the Google Maps API v2 which has been retired/deprecated for a long time (https://developers.google.com/maps/documentation/javascript/versions#documentation-for-the-api-versions)
- Hasn't received a not-cleanup commit since Jul 2013
Change History (5)
comment:1 by , 8 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:2 by , 8 years ago
There is a also bug fix commit in 2014: e7e3435b4d0d409aa98a66d2d7961f7541d19f9f.
While the default version uses version 2 of the Google Maps API, there's a hook for customizing the version so it's not clear to me that the code is completely incompatible with the newest version of the API -- did you test it?
comment:3 by , 8 years ago
| Has patch: | set |
|---|
Note:
See TracTickets
for help on using tickets.
Sure, better remove it than let a completely unmaintained code path.