Skip to content

Commit

Permalink
Updating boolean checks to only suggest Connect for connectors that a…
Browse files Browse the repository at this point in the history
…re not included in SaaS products
  • Loading branch information
jst-cyr committed Mar 18, 2024
1 parent 2c4b38a commit 41df330
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions src/models/OutcomeConditions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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
);
}

Expand Down Expand Up @@ -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);
}

Expand Down

0 comments on commit 41df330

Please sign in to comment.