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

WIP: [8472] migrate point field to GeoDjango to allow filtering by distance #6023

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

goapunk
Copy link
Contributor

@goapunk goapunk commented Jan 30, 2025

Describe your changes
Migrate the point field of kiezradar, project, plans, bplans and extprojects to GeoDjango to allow things like filtering by distance, validate if a point is in the berlin polygon etc. This is quite a big change but I couldn't come up with any other good solution. The actual notification part is still missing though.

depends on liqd/adhocracy4#1765

Tasks

  • PR name contains story or task reference
  • Steps to recreate and test the changes
  • Documentation (docs and inline)
  • Tests (including n+1 and django_assert_num_queries where applicable)
  • Changelog

@goapunk goapunk changed the base branch from main to dev January 30, 2025 18:03
@goapunk goapunk requested review from m4ra and partizipation January 30, 2025 18:03
to allow filtering by distance on a database level.

- add restframework-gis
- add new mixins which handle conversion between geojson and GeoDjango
  fields
- use spatialite instead of sqlite to support gis/GeoDjango
@goapunk goapunk force-pushed the jd-2025-01-point-fields branch from 1253a30 to 291a0a6 Compare February 11, 2025 14:34
@goapunk goapunk changed the title [8472] migrate point field to GeoDjango to allow filtering by distance WIP: [8472] migrate point field to GeoDjango to allow filtering by distance Feb 11, 2025
@m4ra
Copy link
Contributor

m4ra commented Feb 12, 2025

@goapunk I think we need also to enable the extension postgis within a django migration

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

Successfully merging this pull request may close these issues.

2 participants