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

Land Manager account improvements; site-filter refactor; and more... #181

Merged
merged 38 commits into from
Feb 1, 2021

Conversation

mradamcox
Copy link
Contributor

This is a big update to how the site-filter and land manager accounts work together, and it comes along with some other updates as well.

  • All new models are created in the hms app, so the old ones in fpan can be more easily deprecated.
  • Create new Land Manager profiles with more unified and flexible permissions filter settings
  • Create new Management Area and Management Area Group models to consolidate the previous Region and Managed Area models, and allow these to be attached to Land Manager accounts
  • Big overhaul to site-filter permissions logic. Especially, consolidate all previous code into the actual search component file (site-filter.py), and refactor it significantly with an eye toward reusabilty.
  • Support special permissions on multiple resource models (spurred by Public map issue #180) by fixing the iterations and ES clause additions in site-filter.
  • Create new geo filter within site-filter, which uses Management Area (or Management Area Group) geometries from the ORM (which are optionally attached to Land Manager accounts) to filter search results and resource layers on the map.
  • Add management command check_indexes which compares the database resources to the ES index, and attempts to reindex any that are missing
  • Add management command areas with a suite of operations to manage the loading/removal of new Management Areas from shapefiles/geopackages.
  • Add operation to above command that migrates all of the existing fpan.models.ManagedAreas (300) and Regions (8) into the new hms.models.ManagementArea model so they can uniformly be used for filtering.

Tickets addressed:
#180 (closed)
#159 (partial)
#127 (closed)
#173 (closed)

Tickets to re-evaluate now that these updates are in place:
#32, #61, #63, #126

Signed-off-by: Adam Cox <[email protected]>
@mradamcox mradamcox merged commit 142d948 into master Feb 1, 2021
@mradamcox mradamcox deleted the land_manager_update branch February 1, 2021 17:09
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.

1 participant