From 059b635f47f28f1e198cf35b094e8a2fbe741275 Mon Sep 17 00:00:00 2001
From: Severin Beauvais <severin.beauvais@gmail.com>
Date: Tue, 5 Dec 2023 16:33:12 -0800
Subject: [PATCH 1/3] - app version = 5.6.10 - updated some validity
 calculations - moved ting validations to a mixin - generalized status
 computation - updated regular amalgation validity

---
 package-lock.json                             |   4 +-
 package.json                                  |   4 +-
 .../Amalgamation/AmalgamatingBusinesses.vue   |  20 ++-
 src/components/Amalgamation/BusinessTable.vue |  80 +++---------
 .../amalgamation-state-interface.ts           |   2 +-
 src/mixins/amalgamation-mixin.ts              | 114 ++++++++++++++++++
 src/mixins/index.ts                           |   1 +
 src/store/store.ts                            |   9 +-
 src/views/AmalgamationRegular/Information.vue |  12 +-
 .../AmalgamationRegular/ReviewConfirm.vue     |   6 +-
 .../IncorporationDefineCompany.vue            |   2 +-
 11 files changed, 171 insertions(+), 83 deletions(-)
 create mode 100644 src/mixins/amalgamation-mixin.ts

diff --git a/package-lock.json b/package-lock.json
index 9c6804b59..d7da19105 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "business-create-ui",
-  "version": "5.6.9",
+  "version": "5.6.10",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "business-create-ui",
-      "version": "5.6.9",
+      "version": "5.6.10",
       "dependencies": {
         "@babel/compat-data": "^7.21.5",
         "@bcrs-shared-components/approval-type": "1.0.19",
diff --git a/package.json b/package.json
index b8580fdd4..04a0645d3 100644
--- a/package.json
+++ b/package.json
@@ -1,13 +1,13 @@
 {
   "name": "business-create-ui",
-  "version": "5.6.9",
+  "version": "5.6.10",
   "private": true,
   "appName": "Create UI",
   "sbcName": "SBC Common Components",
   "scripts": {
     "dev": "vite",
     "build": "vite build",
-    "serve": "vite preview",
+    "serve": "vite preview --port 8080",
     "lint": "eslint . --ext .js,.ts,.vue",
     "test:unit": "vitest run",
     "test:coverage": "vitest run --coverage"
diff --git a/src/components/Amalgamation/AmalgamatingBusinesses.vue b/src/components/Amalgamation/AmalgamatingBusinesses.vue
index 0d2131f41..e8dcae4b3 100644
--- a/src/components/Amalgamation/AmalgamatingBusinesses.vue
+++ b/src/components/Amalgamation/AmalgamatingBusinesses.vue
@@ -127,8 +127,6 @@
       </ul>
     </v-row> -->
 
-    <!-- FOR DEBUGGING -->
-    <!-- <pre>getAmalgamatingBusinesses={{ getAmalgamatingBusinesses }}</pre> -->
     <BusinessTable
       class="mt-8"
       :class="{ 'invalid-section': getShowErrors && !businessTableValid }"
@@ -138,7 +136,7 @@
 </template>
 
 <script lang="ts">
-import { Component, Mixins } from 'vue-property-decorator'
+import { Component, Mixins, Watch } from 'vue-property-decorator'
 import { Action, Getter } from 'pinia-class'
 import { useStore } from '@/store/store'
 import { CommonMixin } from '@/mixins'
@@ -161,11 +159,10 @@ export default class AmalgamatingBusinesses extends Mixins(CommonMixin) {
   @Getter(useStore) isAmalgamationFilingHorizontal!: boolean
   @Getter(useStore) isRoleStaff!: boolean
 
-  @Action(useStore) setAmalgamatingBusinesses!: (x: Array<any>) => void
+  @Action(useStore) setAmalgamatingBusinesses!: (x: Array<AmalgamatingBusinessIF>) => void
   @Action(useStore) setAmalgamatingBusinessesValid!: (x: boolean) => void
 
   // Local properties
-  amalgamatingBusinessesValid = false
   amalgamatingBusinesses = []
   initialBusinessLookupObject = EmptyBusinessLookup
   businessTableValid = false
@@ -247,6 +244,18 @@ export default class AmalgamatingBusinesses extends Mixins(CommonMixin) {
     this.isAddingAmalgamatingBusiness = false
     this.setAmalgamatingBusinessesValid(true)
   }
+
+  /** Sets validity according to various flags. */
+  @Watch('businessTableValid')
+  @Watch('isAddingAmalgamatingBusiness')
+  @Watch('isAddingAmalgamatingForeignBusiness')
+  private onBusinessTableValid (): void {
+    this.setAmalgamatingBusinessesValid(
+      this.businessTableValid &&
+      !this.isAddingAmalgamatingBusiness &&
+      !this.isAddingAmalgamatingForeignBusiness
+    )
+  }
 }
 </script>
 
@@ -256,5 +265,4 @@ export default class AmalgamatingBusinesses extends Mixins(CommonMixin) {
 .v-btn:not(.v-btn--round).v-size--default {
   height: 44px;
 }
-
 </style>
diff --git a/src/components/Amalgamation/BusinessTable.vue b/src/components/Amalgamation/BusinessTable.vue
index ca49141ba..071320ae5 100644
--- a/src/components/Amalgamation/BusinessTable.vue
+++ b/src/components/Amalgamation/BusinessTable.vue
@@ -69,15 +69,16 @@
 </template>
 
 <script lang="ts">
-import { Component, Emit, Vue, Watch } from 'vue-property-decorator'
-import { Getter, Action } from 'pinia-class'
+import { Component, Emit, Mixins, Watch } from 'vue-property-decorator'
+import { Action } from 'pinia-class'
 import { getName } from 'country-list'
 import { useStore } from '@/store/store'
 import { AmalgamatingStatuses, AmlRoles } from '@/enums'
 import { AmalgamatingBusinessIF } from '@/interfaces'
 import { BaseAddress } from '@bcrs-shared-components/base-address'
 import BusinessStatus from './BusinessStatus.vue'
-import { CorpTypeCd, GetCorpFullDescription } from '@bcrs-shared-components/corp-type-module'
+import { GetCorpFullDescription } from '@bcrs-shared-components/corp-type-module'
+import { AmalgamationMixin } from '@/mixins'
 
 @Component({
   components: {
@@ -85,75 +86,32 @@ import { CorpTypeCd, GetCorpFullDescription } from '@bcrs-shared-components/corp
     BusinessStatus
   }
 })
-export default class BusinessTable extends Vue {
+export default class BusinessTable extends Mixins(AmalgamationMixin) {
   readonly AmlRoles = AmlRoles
   readonly GetCorpFullDescription = GetCorpFullDescription
 
-  @Getter(useStore) getAmalgamatingBusinesses!: AmalgamatingBusinessIF[]
-  @Getter(useStore) isRoleStaff!: boolean
-  @Getter(useStore) isTypeBcCcc!: boolean
-  @Getter(useStore) isTypeBcUlcCompany!: boolean
-
   @Action(useStore) setAmalgamatingBusinesses!: (x: AmalgamatingBusinessIF[]) => void
   @Action(useStore) setDefineCompanyStepValidity!: (x: boolean) => void
 
-  // *** I'M STILL WONDERING IF I WANT TO USE THESE
-  // readonly isLear = (item: AmalgamatingBusinessIF): boolean => (item?.type === 'lear')
-  // readonly isForeign = (item: AmalgamatingBusinessIF): boolean => (item?.type === 'foreign')
-
-  /** True if there a limited company in the table. */
-  get isAnyLimited (): boolean {
-    return this.businesses.some(business =>
-      (business.type === 'lear' && business.legalType === CorpTypeCd.BC_COMPANY)
-    )
-  }
-
   /**
    * This is the list of amalgamating businesses with computed statuses.
-   * In other words, these are the business rules.
+   * In other words, this is where the business rules are evaluated.
    */
   get businesses (): AmalgamatingBusinessIF[] {
     return this.getAmalgamatingBusinesses.map(business => {
-      /* eslint-disable brace-style */
-
-      // disallow foreign altogether if not staff
-      // (could happen if staff added it and regular user resumes draft)
-      if (business.type === 'foreign' && !this.isRoleStaff) {
-        business.status = AmalgamatingStatuses.ERROR_FOREIGN
-      }
-
-      // disallow foreign into ULC if there is also a limited
-      else if (business.type === 'foreign' && this.isTypeBcUlcCompany && this.isAnyLimited) {
-        business.status = AmalgamatingStatuses.ERROR_FOREIGN
-      }
-
-      // assume business is not affiliated if we don't have address (non-staff only)
-      else if (business.type === 'lear' && !business.address && !this.isRoleStaff) {
-        business.status = AmalgamatingStatuses.ERROR_AFFILIATION
-      }
-
-      // identify CCC mismatch
-      else if (business.type === 'lear' && business.legalType === CorpTypeCd.BC_CCC && !this.isTypeBcCcc) {
-        business.status = AmalgamatingStatuses.ERROR_CCC_MISMATCH
-      }
-
-      // disallow NIGS if not staff
-      else if (business.type === 'lear' && !business.goodStanding && !this.isRoleStaff) {
-        business.status = AmalgamatingStatuses.ERROR_NIGS
-      }
-
-      // check if limited restoration
-      // *** TODO
-
-      // check for future effective filing
-      // *** TODO
-
-      // otherwise, status is OK
-      else {
-        business.status = AmalgamatingStatuses.OK
-      }
-      /* eslint-enable brace-style */
-
+      // evaluate the rules for the current business
+      // assign the value of the first failed rule (if any) else OK
+      business.status = this.rules.reduce(
+        (status: AmalgamatingStatuses, rule: (business: AmalgamatingBusinessIF) => AmalgamatingStatuses) => {
+          // if we already failed a rule, don't check the rest of the rules
+          if (status) return status
+          // return the value of the current rule (may be null)
+          return rule(business)
+        },
+        null
+      ) || AmalgamatingStatuses.OK
+
+      // return updated business object
       return business
     })
   }
diff --git a/src/interfaces/store-interfaces/state-interfaces/amalgamation-state-interface.ts b/src/interfaces/store-interfaces/state-interfaces/amalgamation-state-interface.ts
index 3611d651f..cd8bec43c 100644
--- a/src/interfaces/store-interfaces/state-interfaces/amalgamation-state-interface.ts
+++ b/src/interfaces/store-interfaces/state-interfaces/amalgamation-state-interface.ts
@@ -42,7 +42,7 @@ export type AmalgamatingBusinessIF = AmalgamatingLearIF | AmalgamatingForeignIF
 /** State interface for amalgamation-specific data. */
 export interface AmalgamationStateIF {
   amalgamatingBusinesses: Array<AmalgamatingBusinessIF>
-  courtApproval: boolean
   amalgamatingBusinessesValid: boolean
+  courtApproval: boolean
   type: AmalgamationTypes
 }
diff --git a/src/mixins/amalgamation-mixin.ts b/src/mixins/amalgamation-mixin.ts
new file mode 100644
index 000000000..61962b52c
--- /dev/null
+++ b/src/mixins/amalgamation-mixin.ts
@@ -0,0 +1,114 @@
+import { Component, Vue } from 'vue-property-decorator'
+import { Getter } from 'pinia-class'
+import { useStore } from '@/store/store'
+import { AmalgamatingStatuses } from '@/enums'
+import { AmalgamatingBusinessIF } from '@/interfaces'
+import { CorpTypeCd } from '@bcrs-shared-components/corp-type-module'
+
+/**
+ * Mixin that provides amalgamation rules, etc.
+ */
+@Component({})
+export default class AmalgamationMixin extends Vue {
+  @Getter(useStore) getAmalgamatingBusinesses!: AmalgamatingBusinessIF[]
+  @Getter(useStore) isRoleStaff!: boolean
+  @Getter(useStore) isTypeBcCcc!: boolean
+  @Getter(useStore) isTypeBcUlcCompany!: boolean
+
+  // *** TODO: finish this, maybe name them something recognizable...
+  /** Iterable array of rule functions, sorted by importance. */
+  readonly rules = [
+    this.rule1,
+    this.rule2,
+    this.rule3,
+    this.rule4,
+    this.rule5,
+    this.rule6
+  ]
+
+  // *** I'M STILL WONDERING IF I WANT TO USE THESE
+  // readonly isLear = (item: AmalgamatingBusinessIF): boolean => (item?.type === 'lear')
+  // readonly isForeign = (item: AmalgamatingBusinessIF): boolean => (item?.type === 'foreign')
+
+  /** True if there a limited company in the table. */
+  get isAnyLimited (): boolean {
+    return this.getAmalgamatingBusinesses.some(business =>
+      (business.type === 'lear' && business.legalType === CorpTypeCd.BC_COMPANY)
+    )
+  }
+
+  /** True if there an unlimited company in the table in Alberta, Nova Scotia or USA. */
+  get isAnyUnlimitedAbNsUsa (): boolean {
+    return this.getAmalgamatingBusinesses.some(business =>
+      (business.type === 'lear' && business.legalType === CorpTypeCd.BC_COMPANY)
+    )
+  }
+
+  // readonly amalgamationRules = [
+  //   {
+  //     id: 0,
+  //     rule: (v: any) => !!v || 'Required',
+  //     status: AmalgamatingStatuses.ERROR_FOREIGN
+  //   }
+  // ]
+
+  // A BC Company cannot amalgamate with an existing Unlimited Liability Company from Alberta,
+  // Nova Scotia, or the USA to form a BC Unlimited Liability Company.
+
+  // disallow foreign into ULC if there is also a limited
+  rule1 (business: AmalgamatingBusinessIF): AmalgamatingStatuses {
+    if (business.type === 'foreign' && this.isTypeBcUlcCompany && this.isAnyLimited) {
+      return AmalgamatingStatuses.ERROR_FOREIGN
+    }
+    return null
+  }
+
+  // disallow foreign altogether if not staff
+  // (could happen if staff added it and regular user resumes draft)
+  rule2 (business: AmalgamatingBusinessIF): AmalgamatingStatuses {
+    if (business.type === 'foreign' && !this.isRoleStaff) {
+      return AmalgamatingStatuses.ERROR_FOREIGN
+    }
+    return null
+  }
+
+  // disallow foreign into ULC if there is also a limited
+  rule3 (business: AmalgamatingBusinessIF): AmalgamatingStatuses {
+    if (business.type === 'foreign' && this.isTypeBcUlcCompany && this.isAnyLimited) {
+      return AmalgamatingStatuses.ERROR_FOREIGN
+    }
+    return null
+  }
+
+  // assume business is not affiliated if we don't have address (non-staff only)
+  rule4 (business: AmalgamatingBusinessIF): AmalgamatingStatuses {
+    // *** TODO: revert staff check
+    if (business.type === 'lear' && !business.address /* && !this.isRoleStaff */) {
+      return AmalgamatingStatuses.ERROR_AFFILIATION
+    }
+    return null
+  }
+
+  // identify CCC mismatch
+  rule5 (business: AmalgamatingBusinessIF): AmalgamatingStatuses {
+    if (business.type === 'lear' && business.legalType === CorpTypeCd.BC_CCC && !this.isTypeBcCcc) {
+      return AmalgamatingStatuses.ERROR_CCC_MISMATCH
+    }
+    return null
+  }
+
+  // disallow NIGS if not staff
+  rule6 (business: AmalgamatingBusinessIF): AmalgamatingStatuses {
+    // *** TODO: revert staff check
+    if (business.type === 'lear' && !business.goodStanding /* && !this.isRoleStaff */) {
+      return AmalgamatingStatuses.ERROR_NIGS
+    }
+    return null
+  }
+
+  // check if limited restoration
+  // *** TODO
+
+  // check for future effective filing
+  // *** TODO
+}
diff --git a/src/mixins/index.ts b/src/mixins/index.ts
index 145b51cdb..0026789fc 100644
--- a/src/mixins/index.ts
+++ b/src/mixins/index.ts
@@ -1,4 +1,5 @@
 export { default as AddEditOrgPersonMixin } from './add-edit-org-person-mixin'
+export { default as AmalgamationMixin } from './amalgamation-mixin'
 export { default as CommonMixin } from './common-mixin'
 export { default as CurrencyLookupMixin } from '@/mixins/currency-lookup-mixin'
 export { default as DateMixin } from './date-mixin'
diff --git a/src/store/store.ts b/src/store/store.ts
index 2bd797d64..a6887d5c4 100644
--- a/src/store/store.ts
+++ b/src/store/store.ts
@@ -574,8 +574,13 @@ export const useStore = defineStore('store', {
 
     /** Whether all the amalgamation (regular) steps are valid. */
     isAmalgamationRegularValid (): boolean {
-      // *** TODO: implement this
-      return false
+      // *** TODO: add checks for review page components
+      return (
+        this.getAmalgamatingBusinessesValid &&
+        this.getDefineCompanyStep.valid &&
+        this.getAddPeopleAndRoleStep.valid &&
+        this.getCreateShareStructureStep.valid
+      )
     },
 
     /** Whether all the incorporation steps are valid. */
diff --git a/src/views/AmalgamationRegular/Information.vue b/src/views/AmalgamationRegular/Information.vue
index d10e25208..bfe706dbc 100644
--- a/src/views/AmalgamationRegular/Information.vue
+++ b/src/views/AmalgamationRegular/Information.vue
@@ -46,7 +46,6 @@
 import { Component, Mixins, Watch } from 'vue-property-decorator'
 import { Getter, Action } from 'pinia-class'
 import { useStore } from '@/store/store'
-import { DefineCompanyIF } from '@/interfaces'
 import { CommonMixin } from '@/mixins'
 import { RouteNames } from '@/enums'
 import { ExpandableHelp } from '@bcrs-shared-components/expandable-help'
@@ -63,10 +62,9 @@ import AmalgamatingBusinesses from '@/components/Amalgamation/AmalgamatingBusine
   }
 })
 export default class AmalgamationRegularInformation extends Mixins(CommonMixin) {
-  @Getter(useStore) getDefineCompanyStep!: DefineCompanyIF
+  @Getter(useStore) getAmalgamatingBusinessesValid!: boolean
   @Getter(useStore) getNameTranslationsValid!: boolean
   @Getter(useStore) getShowErrors!: boolean
-  @Getter(useStore) isBaseCompany!: boolean
 
   @Action(useStore) setDefineCompanyStepValidity!: (x: boolean) => void
   @Action(useStore) setIgnoreChanges!: (x: boolean) => void
@@ -83,8 +81,8 @@ export default class AmalgamationRegularInformation extends Mixins(CommonMixin)
   /** Object of valid flags. Must match validComponents above. */
   get validFlags (): object {
     return {
-      validAmalgamatingBusinesses: this.amalgamatingBusinessesValid,
-      validNameTranslation: this.getNameTranslationsValid
+      validAmalgamatingBusinesses: this.getAmalgamatingBusinessesValid,
+      validNameTranslations: this.getNameTranslationsValid
     }
   }
 
@@ -99,8 +97,8 @@ export default class AmalgamationRegularInformation extends Mixins(CommonMixin)
     })
   }
 
-  onAmalgamatingBusinessesValid (valid: boolean): void {
-    this.amalgamatingBusinessesValid = valid
+  @Watch('getAmalgamatingBusinessesValid')
+  private onAmalgamatingBusinessesValid (): void {
     this.setDefineCompanyStepValidity(
       this.amalgamatingBusinessesValid &&
       this.getNameTranslationsValid
diff --git a/src/views/AmalgamationRegular/ReviewConfirm.vue b/src/views/AmalgamationRegular/ReviewConfirm.vue
index 157d21771..7b195bd55 100644
--- a/src/views/AmalgamationRegular/ReviewConfirm.vue
+++ b/src/views/AmalgamationRegular/ReviewConfirm.vue
@@ -33,7 +33,10 @@
           icon="mdi-domain"
           label="Amalgamating Businesses Information"
         />
-        <div class="pa-4">
+        <div
+          class="pa-4"
+          :class="{ 'invalid-section': !getAmalgamatingBusinessesValid }"
+        >
           <BusinessTableSummary />
         </div>
       </v-card>
@@ -277,6 +280,7 @@ import { FilingNames } from '@bcrs-shared-components/enums'
   }
 })
 export default class AmalgamationRegularReviewConfirm extends Vue {
+  @Getter(useStore) getAmalgamatingBusinessesValid!: boolean
   @Getter(useStore) getBusinessContact!: ContactPointIF
   @Getter(useStore) getCertifyState!: CertifyIF
   @Getter(useStore) getCourtOrderStep!: CourtOrderStepIF
diff --git a/src/views/Incorporation/IncorporationDefineCompany.vue b/src/views/Incorporation/IncorporationDefineCompany.vue
index 8725d7335..341c66520 100644
--- a/src/views/Incorporation/IncorporationDefineCompany.vue
+++ b/src/views/Incorporation/IncorporationDefineCompany.vue
@@ -237,7 +237,7 @@ export default class IncorporationDefineCompany extends Mixins(CommonMixin) {
   /** Object of valid flags. Must match validComponents. */
   get validFlags (): object {
     return {
-      validNameTranslation: this.isTypeCoop ? true : this.getNameTranslationsValid,
+      validNameTranslations: this.isTypeCoop ? true : this.getNameTranslationsValid,
       validCooperativeType: this.isTypeCoop ? this.hasValidCooperativeType : true,
       validAddressForm: this.addressFormValid,
       validBusinessContactForm: this.businessContactFormValid

From a77986cf108f82b70cf9179a7139190e843b245f Mon Sep 17 00:00:00 2001
From: Severin Beauvais <severin.beauvais@gmail.com>
Date: Thu, 7 Dec 2023 13:56:43 -0800
Subject: [PATCH 2/3] - updated to use temp copy of array (instead of local
 copy)

---
 .../Amalgamation/AmalgamatingBusinesses.vue   | 20 ++++++++-----------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/src/components/Amalgamation/AmalgamatingBusinesses.vue b/src/components/Amalgamation/AmalgamatingBusinesses.vue
index e8dcae4b3..0e26540e1 100644
--- a/src/components/Amalgamation/AmalgamatingBusinesses.vue
+++ b/src/components/Amalgamation/AmalgamatingBusinesses.vue
@@ -106,7 +106,7 @@
       <ul>
         Amalgamating Businesses: <br><br>
         <li
-          v-for="(business, index) in amalgamatingBusinesses"
+          v-for="(business, index) in getAmalgamatingBusinesses"
           :key="index"
         >
           <template v-if="business.foundingDate">
@@ -163,7 +163,6 @@ export default class AmalgamatingBusinesses extends Mixins(CommonMixin) {
   @Action(useStore) setAmalgamatingBusinessesValid!: (x: boolean) => void
 
   // Local properties
-  amalgamatingBusinesses = []
   initialBusinessLookupObject = EmptyBusinessLookup
   businessTableValid = false
 
@@ -173,11 +172,6 @@ export default class AmalgamatingBusinesses extends Mixins(CommonMixin) {
 
   readonly BusinessLookupServices = BusinessLookupServices
 
-  // If continuing a draft, initialize the amalgamatingBusinesses array as the previously saved one.
-  mounted (): void {
-    if (this.getAmalgamatingBusinesses) this.amalgamatingBusinesses = this.getAmalgamatingBusinesses
-  }
-
   // Cancel button in "Add an Amalgamating Business" is pressed.
   addAmalgamatingBusinessCancel (): void {
     this.isAddingAmalgamatingBusiness = false
@@ -221,6 +215,8 @@ export default class AmalgamatingBusinesses extends Mixins(CommonMixin) {
     // If the amalgamating businesses array is not empty, check if identifier already exists.
     // If identifier already exists, don't add the business to the array.
     if (business) {
+      const amalgamatingBusinesses = this.getAmalgamatingBusinesses
+
       const tingBusiness = {
         type: 'lear',
         role: AmlRoles.AMALGAMATING,
@@ -232,14 +228,14 @@ export default class AmalgamatingBusinesses extends Mixins(CommonMixin) {
         goodStanding: business.goodStanding
       } as AmalgamatingBusinessIF
 
-      if (!this.amalgamatingBusinesses.find(b => b.identifier === business.identifier)) {
-        this.amalgamatingBusinesses.push(tingBusiness)
+      if (!amalgamatingBusinesses.find((b: any) => b.identifier === business.identifier)) {
+        amalgamatingBusinesses.push(tingBusiness)
+
+        // Set the new amalgamated businesses array in the store.
+        this.setAmalgamatingBusinesses(amalgamatingBusinesses)
       }
     }
 
-    // Set the amalgamated businesses array in the store.
-    this.setAmalgamatingBusinesses(this.amalgamatingBusinesses)
-
     // Close the "Add an Amalgamating Business" Panel.
     this.isAddingAmalgamatingBusiness = false
     this.setAmalgamatingBusinessesValid(true)

From b55a7ce9c77f959091e3ded5291f6455e77d3ef7 Mon Sep 17 00:00:00 2001
From: Severin Beauvais <severin.beauvais@gov.bc.ca>
Date: Sat, 9 Dec 2023 11:44:13 -0800
Subject: [PATCH 3/3] wip

---
 src/mixins/amalgamation-mixin.ts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/mixins/amalgamation-mixin.ts b/src/mixins/amalgamation-mixin.ts
index 61962b52c..d9f1e79c7 100644
--- a/src/mixins/amalgamation-mixin.ts
+++ b/src/mixins/amalgamation-mixin.ts
@@ -108,7 +108,9 @@ export default class AmalgamationMixin extends Vue {
 
   // check if limited restoration
   // *** TODO
+  // *** need to look at all business' filings
 
   // check for future effective filing
   // *** TODO
+  // *** need to look at all business' filings
 }