Skip to content

PSP-11077: Unable to edit properties on a file when one or more property is retired#5164

Merged
areyeslo merged 3 commits intobcgov:devfrom
areyeslo:PSP-11077-UnableEditPropertiesWithRetiredProperties
Jan 21, 2026
Merged

PSP-11077: Unable to edit properties on a file when one or more property is retired#5164
areyeslo merged 3 commits intobcgov:devfrom
areyeslo:PSP-11077-UnableEditPropertiesWithRetiredProperties

Conversation

@areyeslo
Copy link
Collaborator

@areyeslo areyeslo commented Jan 16, 2026

  • Only block new retired properties from being added.

  • Allow edits/removal of already-attached retired properties.

  • Validation is implemented in the backend which handles complex validations instead of frontend.
    Note: When implementing customized validation, Formik’s error state is updated after the next render cycle. Then, found "validates form values upon submission" test failing because the text Please check form fields for errors was not present in the DOM immediately after click, the error state may not have propagated yet, so the error message is not rendered causing the test to fail.

Story

@areyeslo areyeslo self-assigned this Jan 16, 2026
@areyeslo areyeslo added the enhancement New feature or request label Jan 16, 2026
@github-actions
Copy link
Contributor

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

@areyeslo areyeslo marked this pull request as draft January 16, 2026 17:27
@areyeslo areyeslo force-pushed the PSP-11077-UnableEditPropertiesWithRetiredProperties branch from 3058dbf to 9d41b24 Compare January 16, 2026 21:13
@github-actions
Copy link
Contributor

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

1 similar comment
@github-actions
Copy link
Contributor

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

@areyeslo areyeslo force-pushed the PSP-11077-UnableEditPropertiesWithRetiredProperties branch from 9d41b24 to f48110c Compare January 16, 2026 21:39
@github-actions
Copy link
Contributor

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

@areyeslo areyeslo force-pushed the PSP-11077-UnableEditPropertiesWithRetiredProperties branch from f48110c to 4389575 Compare January 16, 2026 22:44
@github-actions
Copy link
Contributor

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

@areyeslo areyeslo force-pushed the PSP-11077-UnableEditPropertiesWithRetiredProperties branch from 4389575 to 0797d98 Compare January 16, 2026 23:09
@github-actions
Copy link
Contributor

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

1 similar comment
@github-actions
Copy link
Contributor

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

@areyeslo areyeslo marked this pull request as ready for review January 16, 2026 23:33
@sonarqubecloud
Copy link

@github-actions
Copy link
Contributor

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

1 similar comment
@github-actions
Copy link
Contributor

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

@areyeslo areyeslo added this pull request to the merge queue Jan 21, 2026
Merged via the queue into bcgov:dev with commit 0c19c43 Jan 21, 2026
18 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