Opened 10 years ago
Closed 10 years ago
#23781 closed New feature (fixed)
Add a GeoJSON serializer
Reported by: | Claude Paroz | Owned by: | nobody |
---|---|---|---|
Component: | GIS | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
I think outputting GeoJSON data is very common these days, and I think Django should provide a serializer to that purpose. It is not meant to replace the django-geojson package which has much more than that. Instead, my proposal would be to add a simple serializer (based on the json one) that people (and maybe django-geojson) can subclass and extend, if needed.
Change History (5)
comment:1 by , 10 years ago
Has patch: | set |
---|
comment:2 by , 10 years ago
Patch needs improvement: | set |
---|---|
Triage Stage: | Unreviewed → Accepted |
Tests are not passing.
comment:3 by , 10 years ago
Patch needs improvement: | unset |
---|
Fixed. Sorry, the first patch was definitely not ready.
comment:4 by , 10 years ago
(Working at the django-under-the-hood sprint)
I looked at the pull request: looks good. A specific geojson serializer is handy and fits with geodjango.
I added one comment to the pull request with a small documentation re-wording suggestion.
comment:5 by , 10 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
In 35dac5070bc3422cd2c6c2f46a28d1a1af2a2c50:
Added a new GeoJSON serialization format for GeoDjango
Thanks Reinout van Rees for the review.
https://github.com/django/django/pull/3487