Skip to content

Conversation

@dimak1
Copy link
Collaborator

@dimak1 dimak1 commented Dec 12, 2025

Issue:

Description of changes:

  • Fix Step 4 Validation of the Host form that was caused by the BL Exp Date
  • Make BL Exp Date validation optional depending on BL Exemption checkbox

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 Dec 12, 2025
@dimak1 dimak1 added UI Host Bugfix Fix for an issue or a bug labels Dec 12, 2025
@sonarqubecloud
Copy link

@bcregistry-sre
Copy link
Collaborator

Temporary Url for review: https://strr-hosts-dev--pr-959-b2jw2ngi.web.app

@dimak1 dimak1 merged commit c2b3008 into bcgov:main Dec 15, 2025
14 checks passed
@dimak1 dimak1 deleted the fix/bl-validation branch December 15, 2025 16:27
dimak1 added a commit that referenced this pull request Dec 16, 2025
* Renewal Period Closed ToDo (#957)

* feat(ui): renewal period closed todo for expired strata and platform registrations

* chore: Remove Renewal Period Closed from Platforms

* fix: remove pr doc requirement if host is not tenant (#958)

* Fix Host Validation (#959)

* fix(ui): host review step 4 validation

* chore: host version update

* chore: revert small change

* chore: comment cleanup
dimak1 added a commit that referenced this pull request Dec 22, 2025
* Renewal Period Closed ToDo (#957)

* feat(ui): renewal period closed todo for expired strata and platform registrations

* chore: Remove Renewal Period Closed from Platforms

* fix: remove pr doc requirement if host is not tenant (#958)

* Fix Host Validation (#959)

* fix(ui): host review step 4 validation

* chore: host version update

* chore: revert small change

* chore: comment cleanup

* 26715 - Fix Hosts system preventing to add property manager (#963)

* fix: update the version of the API or flask-jwt-oidc (#966)

* chore: cleanup unused strr-web app and related references (#968)

* chore: update package versions in base layer (#964)

* Update Package Versions in Platform app (#961)

* chore: update package versions for platform app

* chore: update version of platform app itself

* ci: skip sonarqube analysis of strr-web folder

* refactor: remove fr-CA translation file from platform because same file exists in base later already

* Update package versions in Strata app (#967)

* chore: update package versions in strata app

* fix: test mocks in Host app

---------

Co-authored-by: Karim El Jazzar <122301442+JazzarKarim@users.noreply.github.com>
Co-authored-by: thor <1042854+thorwolpert@users.noreply.github.com>
dimak1 added a commit to dimak1/STRR that referenced this pull request Dec 23, 2025
* Renewal Period Closed ToDo (bcgov#957)

* feat(ui): renewal period closed todo for expired strata and platform registrations

* chore: Remove Renewal Period Closed from Platforms

* fix: remove pr doc requirement if host is not tenant (bcgov#958)

* Fix Host Validation (bcgov#959)

* fix(ui): host review step 4 validation

* chore: host version update

* chore: revert small change

* chore: comment cleanup

* 26715 - Fix Hosts system preventing to add property manager (bcgov#963)

* fix: update the version of the API or flask-jwt-oidc (bcgov#966)

* chore: cleanup unused strr-web app and related references (bcgov#968)

* chore: update package versions in base layer (bcgov#964)

* Update Package Versions in Platform app (bcgov#961)

* chore: update package versions for platform app

* chore: update version of platform app itself

* ci: skip sonarqube analysis of strr-web folder

* refactor: remove fr-CA translation file from platform because same file exists in base later already

* Update package versions in Strata app (bcgov#967)

* chore: update package versions in strata app

* fix: test mocks in Host app

---------

Co-authored-by: Karim El Jazzar <122301442+JazzarKarim@users.noreply.github.com>
Co-authored-by: thor <1042854+thorwolpert@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bugfix Fix for an issue or a bug Host UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants