Skip to content

33417 - Add Validation for International Region in Foreign Juridiction#4391

Merged
meawong merged 1 commit into
bcgov:mainfrom
meawong:33412-Add-Region-Not-Allowed-Validation-for-International-Foreign-Juridiction
May 14, 2026
Merged

33417 - Add Validation for International Region in Foreign Juridiction#4391
meawong merged 1 commit into
bcgov:mainfrom
meawong:33412-Add-Region-Not-Allowed-Validation-for-International-Foreign-Juridiction

Conversation

@meawong
Copy link
Copy Markdown
Collaborator

@meawong meawong commented May 13, 2026

Issue #: /bcgov/entity#33412

Description of changes:

  • Region should not be permitted for international juridictions

This is a common validation and the change will apply to:

  • Amalgamation Application

  • Consent to Amalgamate Out

  • Amalgamation Out

  • Consent to Continue Out

  • Continuation Out

  • Continuation In Authorization & Application

Some examples:
Continuation In Authorization:
image

Consent to Amalgamate Out:
image

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the lear license (Apache 2.0).

@meawong meawong self-assigned this May 13, 2026
@sonarqubecloud
Copy link
Copy Markdown

@meawong meawong requested review from kialj876 and loneil May 14, 2026 00:26
Copy link
Copy Markdown
Collaborator

@kialj876 kialj876 left a comment

Choose a reason for hiding this comment

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

LGTM

@meawong meawong requested a review from mruff-aeq May 14, 2026 15:51
Copy link
Copy Markdown
Collaborator

@loneil loneil left a comment

Choose a reason for hiding this comment

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

Good tests too!

@meawong meawong merged commit da3f97d into bcgov:main May 14, 2026
8 checks passed
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.

4 participants