bcgov / BCHeritage

Branch level repository for documentation and product issues.
Apache License 2.0
4 stars 0 forks source link

Remove copy of Arches Core MVT class when upgrading to dev/7.6.x #1022

Open bferguso opened 2 months ago

bferguso commented 2 months ago

As a workaround to allow different map views for administrators and anonymous users, we copied the MVT class from dev/7.6.x back into our configuration as views/mvt_core.py. Once we move to dev/7.6.x, this should be removed. The things that this fixed are:

  1. Allow tile cacheing by user
  2. Increased mvt query performance
  3. Ignore any resource instance permissions if the user is a superuser

The first two points are already in dev/7.6.x. The third one is not. Need to ensure that the get_restricted_instances call in the permission_backend.py file filters by user, not the current method which is any restriction for any user/group.

This is the workaround for #1019