Mathieu Leplatre

Results 354 comments of Mathieu Leplatre
trafficstars

Could you please provide the full stacktrace please ?

I'm sorry I can't spend some time right now. The paste seems to have been purged... Can you try to accomplish the same operations using the default Django Serializers (JSON...

I don't think we should remove Django from dependencies... In order to upgrade django-geojson, you can use `--no-deps` : ``` pip install -U --no-deps django-geojson ```

It should work with lists, but indeed, not sure with generators... https://github.com/makinacorpus/django-geojson/blob/master/djgeojson/serializers.py#L346-L347 Instead of `isinstance(queryset, list)`, we could check something like `hasattr(queryset, '__iter__')`. Don't hesitate to open a pull-request !

Thanks for take care of this! The issue has been here for a while :) Would you mind adding a little test to assert that the behaviour you just fixed...

Hi @dispiste ! Do you need help to write a small ~~fix~~ test ?

Thanks @aboutaaron for your proposition! Yes please! Without test we can't merge this :)

@aboutaaron @dispiste this cannot be merged without tests unfortunately...

I also released django-geojson 1.8.1 with @kevcooper fix about GEOS detection. Let us know :)

It would be nice to propose a fix on the documentation to make it «**obvious**»