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

Save land ownership layer when saving a map #294

Open
rogup opened this issue Dec 12, 2023 · 2 comments
Open

Save land ownership layer when saving a map #294

rogup opened this issue Dec 12, 2023 · 2 comments

Comments

@rogup
Copy link
Collaborator

rogup commented Dec 12, 2023

Description

The land ownership layer's state gets lost (e.g. the highlighted polygons and whether the layer is displayed) when a map is closed. It makes sense for this data to be saved like other layers and map objects.

This would speed up development a lot, when working on issues relating to land ownership. Since currently we have to repeat a lot of steps to get back to our previous state when reloading.

Acceptance Criteria

The land ownership layer returns to where we left off when a saved map is re-opened or the browser refreshes.

@rogup
Copy link
Collaborator Author

rogup commented Dec 12, 2023

Part of this fix is in #295

To save the highlighted properties, a quick way would be to just include all the property data in the map data JSON. But we shouldn't do this, because the JSON could get too big.
Maybe we should store them properly in the backend's database, like we do with drawn markers/polygons. We should be careful here too, to ensure we reduce load on the database and only include necessary data.
e.g. we don't want to duplicate a load of data that's already in the property boundary service's DB. If a user saves thousands of properties e.g. those belonging to the Queen, maybe we should only certain values such as the polygon ID, name, rough coordinates, so they can then be fetched at runtime via the boundary service.

This is a fairly big task so not a priority. Will leave on the backlog.

@rogup rogup removed their assignment Dec 12, 2023
@lin-d-hop
Copy link
Contributor

I expect there may be other side effects to this as well, as there are still clunky UX elements that a map refresh works around. Hopefully we'll have resolved those before we get to this :)

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

No branches or pull requests

2 participants