Skip to content

PSP-11190 : FT-REG: When properties details form is cancelled and then the user wants to edit it again, the edit button doesn't work#5226

Merged
asanchezr merged 3 commits intobcgov:devfrom
eddherrera:psp-11190
Mar 12, 2026

Conversation

@eddherrera
Copy link
Copy Markdown
Collaborator

No description provided.

…n the user wants to edit it again, the edit button doesn't work
@eddherrera eddherrera self-assigned this Feb 25, 2026
@eddherrera eddherrera added the bug Something isn't working label Feb 25, 2026
@github-actions
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Collaborator

@devinleighsmith devinleighsmith left a comment

Choose a reason for hiding this comment

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

When I tested this just now, clicking the modal that pops up to confirm when you have unsaved changes, and then confirming you want to leave did nothing, I had to click cancel again.

Herrera added 2 commits February 27, 2026 12:38
# Conflicts:
#	source/frontend/src/features/mapSideBar/acquisition/AcquisitionContainer.tsx
@eddherrera eddherrera marked this pull request as ready for review February 27, 2026 21:11
@sonarqubecloud
Copy link
Copy Markdown

@eddherrera
Copy link
Copy Markdown
Collaborator Author

When I tested this just now, clicking the modal that pops up to confirm when you have unsaved changes, and then confirming you want to leave did nothing, I had to click cancel again.

@devinleighsmith seems to be working now after @areyeslo changes.

@github-actions
Copy link
Copy Markdown
Contributor

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

@asanchezr asanchezr added this pull request to the merge queue Mar 12, 2026
Merged via the queue into bcgov:dev with commit 2657d8a Mar 12, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants