Skip to content

PSP-10074: Display Retired Properties on Files#5124

Merged
asanchezr merged 2 commits intobcgov:devfrom
areyeslo:PSP-10074-DisplayRetiredProperties
Dec 22, 2025
Merged

PSP-10074: Display Retired Properties on Files#5124
asanchezr merged 2 commits intobcgov:devfrom
areyeslo:PSP-10074-DisplayRetiredProperties

Conversation

@areyeslo
Copy link
Collaborator

@areyeslo areyeslo commented Dec 17, 2025 •
edited
Loading

Added Retired properties across file property lists. When a property has IS_REITIRED flag set in the database, it is shown under "Retired" in the properties section.

In Management File:

Test property 375 is Inactive, but is also Retired. The Retired state has precedence:
image

Selected Properties section :
image

For the remaining files, Retired property was added if property has IS_RETIRED flag set in the database:
image

Story

@github-actions
Copy link
Contributor

github-actions bot commented Dec 17, 2025

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

@areyeslo areyeslo force-pushed the PSP-10074-DisplayRetiredProperties branch from 6d63e5d to ee4f8da Compare December 17, 2025 23:57
@areyeslo areyeslo marked this pull request as draft December 17, 2025 23:57
@github-actions
Copy link
Contributor

github-actions bot commented Dec 18, 2025

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

1 similar comment
@github-actions
Copy link
Contributor

github-actions bot commented Dec 18, 2025

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

@areyeslo areyeslo force-pushed the PSP-10074-DisplayRetiredProperties branch from b876cc8 to 1d9ebda Compare December 18, 2025 00:17
@devinleighsmith
Copy link
Collaborator

devinleighsmith commented Dec 18, 2025

As per the story, please display the text: "Retired" instead of the active/inactive dropdown (in the screenshot it appears you aren't displaying anything in this case).

@github-actions
Copy link
Contributor

github-actions bot commented Dec 18, 2025

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

@areyeslo areyeslo force-pushed the PSP-10074-DisplayRetiredProperties branch from 1d9ebda to f6dc502 Compare December 18, 2025 00:32
@github-actions
Copy link
Contributor

github-actions bot commented Dec 18, 2025

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

@areyeslo areyeslo force-pushed the PSP-10074-DisplayRetiredProperties branch 4 times, most recently from 3ba40b3 to 4f9576a Compare December 18, 2025 18:46
@areyeslo areyeslo force-pushed the PSP-10074-DisplayRetiredProperties branch from 4f9576a to 87e716c Compare December 18, 2025 19:21
@github-actions
Copy link
Contributor

github-actions bot commented Dec 18, 2025

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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 18, 2025

Quality Gate Passed Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@areyeslo areyeslo added the enhancement New feature or request label Dec 18, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 18, 2025

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

@areyeslo areyeslo marked this pull request as ready for review December 18, 2025 21:56
@asanchezr asanchezr added this pull request to the merge queue Dec 22, 2025
Merged via the queue into bcgov:dev with commit d5ee73d Dec 22, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devinleighsmith devinleighsmith devinleighsmith approved these changes

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@areyeslo @devinleighsmith @asanchezr