Skip to content

Automatically enable the survey plan layer when survey plan search se…#5210

Merged
asanchezr merged 4 commits intobcgov:devfrom
devinleighsmith:psp-11194
Feb 19, 2026
Merged

Automatically enable the survey plan layer when survey plan search se…#5210
asanchezr merged 4 commits intobcgov:devfrom
devinleighsmith:psp-11194

Conversation

@devinleighsmith
Copy link
Copy Markdown
Collaborator

…lected.

@devinleighsmith devinleighsmith self-assigned this Feb 13, 2026
@devinleighsmith devinleighsmith added the enhancement New feature or request label Feb 13, 2026
Copy link
Copy Markdown
Collaborator

@asanchezr asanchezr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, see comments

Comment on lines +112 to +113
mapMachineBaseMock.setMapLayers.mockClear();
mapMachineBaseMock.activeLayers = new Set(initialEnabledLayers);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This code is yielding the following error. Please take a look:

Error: src/features/properties/filter/PropertyFilter.test.tsx(112,37): error TS2339: Property 'mockClear' does not exist on type '(layers: Set<string>) => void'.
Error: Process completed with exit code 2.

@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/5210

@asanchezr asanchezr added this pull request to the merge queue Feb 19, 2026
github-merge-queue Bot pushed a commit that referenced this pull request Feb 19, 2026
Automatically enable the survey plan layer when survey plan search se…
Merged via the queue into bcgov:dev with commit 13ca838 Feb 19, 2026
15 checks 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.

2 participants