Skip to content

fix(examiner-web): show renewal badge only for latest provisional renewals#1504

Merged
jimmypalelil merged 1 commit intobcgov:mainfrom
jimmypalelil:1120-remove-renewal-badge
Apr 8, 2026
Merged

fix(examiner-web): show renewal badge only for latest provisional renewals#1504
jimmypalelil merged 1 commit intobcgov:mainfrom
jimmypalelil:1120-remove-renewal-badge

Conversation

@jimmypalelil
Copy link
Copy Markdown
Collaborator

Issue:

Note: The requirements were revised for this ticket. See comment for more details.


Description of changes:

Display the renewal badge in registrations table on a registration when the newest linked application is a renewal in PROVISIONALLY_APPROVED status and the registration has no decider.

The requirements were revised see comment


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the BC Registry and Digital Services BSD 3-Clause License

…ewals

Display the registration-row renewal badge when the newest linked application is a renewal in PROVISIONALLY_APPROVED status and the registration has no decider.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 7, 2026

@bcregistry-sre
Copy link
Copy Markdown
Collaborator

Temporary Url for review: https://strr-examiner-dev--pr-1504-2y6owdid.web.app

@jimmypalelil jimmypalelil marked this pull request as ready for review April 7, 2026 20:10
Copy link
Copy Markdown
Collaborator

@dimak1 dimak1 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, thanks for the comments.

@jimmypalelil jimmypalelil merged commit 4e3093b into bcgov:main Apr 8, 2026
14 checks passed
@jimmypalelil jimmypalelil deleted the 1120-remove-renewal-badge branch April 8, 2026 14:39
@dimak1 dimak1 mentioned this pull request Apr 8, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants