search

IMGIITRoorkee / omniport-backend

The Django backend of the one true portal for any and every educational institute
GNU General Public License v3.0
68 stars 22 forks source link

Rewrite `LocationInformation` with PostGIS #14

Closed dhruvkb closed 5 years ago

dhruvkb commented 6 years ago

The LocationInformation generic in the Omniport kernel has been written keeping fundamental PostgreSQL in mind. This can be implemented much more optimally with PostGIS, which is the objective of this particular issue.

dhruvkb commented 5 years ago

GeoDjango installation and configuration can be understood here: https://docs.djangoproject.com/en/2.1/ref/contrib/gis/install/ There will also need to be considerable change to the PostgreSQL container for this to work: https://github.com/IMGIITRoorkee/omniport-docker/

The pros and cons of including such a feature are open to debate although my personal preference seems to be leaning towards pushing this feature only if an overwhelming need presents itself, including but not limited to an app involving maps (like campus activity or campus safety) coming up.

dhruvkb commented 5 years ago

This seems unnecessarily complicated and the implementation is beyond our scope and abilities.