From 41df3305f1555d399aa461860eab9cf5db580101 Mon Sep 17 00:00:00 2001 From: Jason St-Cyr Date: Mon, 18 Mar 2024 12:30:45 -0400 Subject: [PATCH] Updating boolean checks to only suggest Connect for connectors that are not included in SaaS products --- src/models/OutcomeConditions.ts | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/src/models/OutcomeConditions.ts b/src/models/OutcomeConditions.ts index 4489dea..59e2dd7 100644 --- a/src/models/OutcomeConditions.ts +++ b/src/models/OutcomeConditions.ts @@ -255,13 +255,19 @@ export class OutcomeConditions { } /** - * Analyzes current answers to determine if the solution has some form of external system integration + * Analyzes current answers to determine if the solution has some form of external system integration that could be moved to Connect. * Any use of Data Exchange Framework or Sitecore external integration module also qualifies for migration + * Not required for some connectors which are included in other products (such as DAM integration) */ - hasSystemIntegration(): boolean { + requiresConnect(): boolean { return ( this.xpFeaturesUsed.externalDataSystems || - !this.connectorsUsed.none + this.connectorsUsed.customDEF || + this.connectorsUsed.dynamics365commerce || + this.connectorsUsed.dynamics365sales || + this.connectorsUsed.komfo || + this.connectorsUsed.sfcrm || + this.connectorsUsed.sfmc ); } @@ -304,7 +310,7 @@ export class OutcomeConditions { } //If they are using some form of integration, we direct towards migrating to Sitecore Connect - if (this.hasSystemIntegration()) { + if (this.requiresConnect()) { products.push(TargetProduct.connect); }