Skip to content

Feat/Support selectedFeatures for multiple maps concurrently#116

Merged
okdv merged 6 commits intodevelopfrom
feat/semantic-map-urls
Aug 16, 2025
Merged

Feat/Support selectedFeatures for multiple maps concurrently#116
okdv merged 6 commits intodevelopfrom
feat/semantic-map-urls

Conversation

@okdv
Copy link
Copy Markdown
Owner

@okdv okdv commented Aug 16, 2025

Pull Request Checklist:

Please ensure the following points are checked before requesting a review. Asterisk (*) denotes that it is required prior to merge.

  • I have read the CONTRIBUTING.md document. *
  • I have updated the documentation (if applicable).
  • I have added or updated any relevant tests
  • I have run npm run pre-push * (if playwright test failed just notate why below)
  • I merged (or rebased) develop into my branch before pushing *
  • This PR is ready for review and not a draft. *

Description:

Updated store so that is uses a Record Set of selectedFeatures within a Record Set of maps (interactive layers)
add functions to the stores service to make updating nested Records simple
updated map service to properly find the selected layer and selector again (using new store data)
created a svelte store for storing the current interactive layer, it is a store only no localstorage


Related Issues (if any):

closes #108


Type of Change:

Please check the applicable options.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring/Code quality improvement
  • Other (please describe below)

How Has This Been Tested?

wasnt sure of any tests to add atm, may add some when entire feature is done
pre-push


Screenshots (if applicable):


Additional Comments:

@okdv okdv added this to the v1.0.0-beta.5 milestone Aug 16, 2025
@okdv okdv self-assigned this Aug 16, 2025
@okdv okdv added the enhancement New feature or request label Aug 16, 2025
@okdv okdv merged commit 000c91c into develop Aug 16, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant