Skip to content

feat(ui): 1081 - hide address error in host renewals#1617

Merged
dimak1 merged 1 commit intobcgov:mainfrom
dimak1:hide-address-error-renewals
May 7, 2026
Merged

feat(ui): 1081 - hide address error in host renewals#1617
dimak1 merged 1 commit intobcgov:mainfrom
dimak1:hide-address-error-renewals

Conversation

@dimak1
Copy link
Copy Markdown
Collaborator

@dimak1 dimak1 commented May 6, 2026

Issue:

Description of changes:

  • Hide error in Host renewals when address validation returns no found address
Screenshot 2026-05-06 at 14 20 44

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

@dimak1 dimak1 self-assigned this May 6, 2026
@dimak1 dimak1 added UI Host Renewals Registration Renewals labels May 6, 2026
@dimak1 dimak1 changed the title ui: 1081 - hide address error in host renewals feat(ui): 1081 - hide address error in host renewals May 6, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 6, 2026

@bcregistry-sre
Copy link
Copy Markdown
Collaborator

Temporary Url for review: https://strr-hosts-dev--pr-1617-vuqtoybv.web.app

<div class="flex flex-col gap-4" data-testid="property-requirements-section">
<FormDefineYourRentalUnitRequirementsError
v-if="reqStore.propertyReqError.type !== undefined"
v-if="reqStore.propertyReqError.type !== undefined && !isRegistrationRenewal"
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.

no unit tests required for this change?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

No, for this we can add a unit test later for the whole component.

@dimak1 dimak1 merged commit b2456f5 into bcgov:main May 7, 2026
14 checks passed
@dimak1 dimak1 deleted the hide-address-error-renewals branch May 7, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Host Renewals Registration Renewals UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants