Merge pull request #383 from snehaljha-sf/migration_fixes

Migration fixes

Author: snehaljha-sf

messages/assess.json

@@ -16,19 +16,19 @@
   "couldNotTruncate": "Could not truncate %s",
   "couldNotTruncateOmnniProcess": "Could not truncate OmniProcess. Please make sure your OS/IP is not referenced in an OmniScrirpt or Flex Card.",
   "invalidNameTypeSubtypeOrLanguage": "Not a valid Name, Type, SubType or Language",
-  "invalidOrRepeatingOmniscriptElementNames": "Invalid or Repeating Element Names for Same OmniScript",
+  "invalidOrRepeatingOmniscriptElementNames": "Invalid or Repeating Element Names for Same Omniscript",
   "invalidDataRaptorName": "Invalid name, can not have special characters",
-  "duplicatedCardName": "Duplicated Flex Card name",
-  "duplicatedDrName": "Duplicated DataMapper name",
-  "duplicatedOSName": "Duplicated OmniScript name",
+  "duplicatedCardName": "Duplicated Flex Card name %s",
+  "duplicatedDrName": "Duplicated DataMapper name %s",
+  "duplicatedOSName": "Duplicated %s name %s",
   "duplicatedName": "Duplicated name",
-  "errorWhileActivatingOs": "Could not activate OmniScript / Integration Procedure: ",
+  "errorWhileActivatingOs": "Could not activate %s: ",
   "errorWhileActivatingCard": "Could not activate Card: ",
   "errorWhileUploadingCard": "An error ocurred while uploading Card: ",
-  "errorWhileCreatingElements": "An error ocurred while saving OmniScript elements: ",
+  "errorWhileCreatingElements": "An error ocurred while saving %s elements: ",
   "allVersionsDescription": "Migrate all versions of a component",
   "changeMessage": " %s will be changed from %s to %s",
-  "angularOSWarning": " Angular OmniScript will not be migrated, please convert this to LWC based Omniscript",
+  "angularOSWarning": " Angular Omniscript will not be migrated, please convert this to LWC based Omniscript",
   "relatedObjectGA": "Please select the type of component to assess: 'apex' for Apex classes, 'lwc' for Lightning Web Components, 'flexipage' for FlexiPages, or 'apex,lwc,flexipage' if you want to include all types.",
   "apexLwc": "Please select the type of component to assess: 'apex' for Apex classes.",
   "invalidNamespace": "The namespace you have passed is not valid namespace, the valid namespace of your org is ",
@@ -37,11 +37,11 @@
   "invalidRelatedObjectsOption": "Invalid option provided for -r: %s. Valid options are apex, lwc, expsites, flexipage.",
   "formulaSyntaxError": "There was some problem while updating the formula syntax, please check the all the formula's syntax once : %s",
   "errorDuringFlexCardAssessment": "Error during FlexCard assessment",
-  "errorDuringOmniScriptAssessment": "Error during OmniScript assessment",
+  "errorDuringOmniScriptAssessment": "Error during Omniscript assessment",
   "errorDuringGlobalAutoNumberAssessment": "Error during GlobalAutoNumber assessment",
   "processingFlexCard": "Processing FlexCard: %s",
   "processingDataRaptor": "Processing DataMapper: %s",
-  "processingOmniScript": "Processing OmniScript: %s",
+  "processingOmniScript": "Processing Omniscript: %s",
   "processingGlobalAutoNumber": "Processing GlobalAutoNumber: %s",
   "foundDataRaptorsToAssess": "Found %s DataMappers to assess",
   "foundOmniScriptsToAssess": "Found %s %s to assess",
@@ -64,7 +64,7 @@
   "assessedGlobalAutoNumbersCount": "Assessed %s GlobalAutoNumbers",
   "globalAutoNumberAssessmentCompleted": "The assessment for Global Auto Number is complete.",
   "omniScriptAssessment": "%s Assessment",
-  "assessedOmniScriptsCount": "Assessed %s OmniScripts",
+  "assessedOmniScriptsCount": "Assessed %s Omniscripts",
   "assessedIntegrationProceduresCount": "Assessed %s Integration Procedures",
   "omniScriptAssessmentCompleted": "%s assessment completed",
   "startingFlexCardAssessment": "Starting FlexCard assessment",
@@ -95,7 +95,7 @@
   "methodCallBundleNameUpdated": "Method call bundle name will be updated in %s for class %s method %s",
   "cardNameChangeMessage": "Card name will be changed from %s to %s to follow API naming standards",
   "authordNameChangeMessage": "Author name will be changed from %s to %s to follow API naming standards",
-  "omniScriptNameChangeMessage": "OmniScript reference part %s will be changed to %s during migration.",
+  "omniScriptNameChangeMessage": "Omniscript reference part %s will be changed to %s during migration.",
   "dataRaptorNameChangeMessage": "DataMapper reference %s will be changed to %s during migration.",
   "integrationProcedureNameChangeMessage": "Integration Procedure reference %s will be changed to %s during migration.",
   "integrationProcedureManualUpdateMessage": "Integration Procedure reference %s may need manual updates after migration.",

messages/migrate.json

@@ -20,15 +20,15 @@
   "couldNotTruncate": "Could not truncate %s",
   "couldNotTruncateOmnniProcess": "Could not truncate OmniProcess. Please make sure your OS/IP is not referenced in an OmniScrirpt or Flex Card.",
   "invalidNameTypeSubtypeOrLanguage": "Not a valid Name, Type, SubType or Language",
-  "invalidOrRepeatingOmniscriptElementNames": "Invalid or Repeating Element Names for Same OmniScript",
+  "invalidOrRepeatingOmniscriptElementNames": "Invalid or Repeating Element Names for Same Omniscript",
   "invalidDataRaptorName": "Invalid name, can not have special characters",
-  "duplicatedCardName": "Duplicated Flex Card name",
-  "duplicatedDrName": "Duplicated DataMapper name",
-  "duplicatedOSName": "Duplicated OmniScript name",
-  "errorWhileActivatingOs": "Could not activate OmniScript / Integration Procedure: ",
+  "duplicatedCardName": "Duplicated Flex Card name %s",
+  "duplicatedDrName": "Duplicated DataMapper name %s",
+  "duplicatedOSName": "Duplicated %s name %s",
+  "errorWhileActivatingOs": "Could not activate %s: ",
   "errorWhileActivatingCard": "Could not activate Card: ",
   "errorWhileUploadingCard": "An error ocurred while uploading Card: ",
-  "errorWhileCreatingElements": "An error ocurred while saving OmniScript elements: ",
+  "errorWhileCreatingElements": "An error ocurred while saving Omniscript elements: ",
   "allVersionsDescription": "Migrate all versions of a component",
   "relatedObjectGA": "Please select the type of components to migrate: 'apex' for Apex classes, 'lwc' for Lightning Web Components, 'flexipage' for FlexiPages, 'expsites' for Experience Sites, or 'apex,lwc,flexipage,expsites' if you want to include all types.",
   "apexLwc": "Please select the type of components to migrate: 'apex' for Apex classes.",
@@ -104,7 +104,7 @@
   "cardAuthorNameChangeMessage": "Card author name has been modified to fit naming rules: %s",
   "cardNameChangeMessage": "Card name has been modified to fit naming rules: %s",
   "duplicateCardNameMessage": "Potential duplicate: Another card has the same name %s after name cleaning. This may cause conflicts during migration",
-  "angularOmniscriptWarningMessage": " Angular OmniScript will not be migrated, please convert this to LWC based Omniscript",
+  "angularOmniscriptWarningMessage": " Angular Omniscript will not be migrated, please convert this to LWC based Omniscript",
   "apexFileChangesApplied": "Applied changes to Apex class %s",
   "generatingComponentReports": "Generating report for components",
   "generatingRelatedObjectReports": "Generating report for related objects",
@@ -201,15 +201,18 @@
   "creatingNPMConfigFile": "Creating npm configuration file",
   "npmConfigFileCreated": "Npm config file created",
   "authKeyEnvVarNotSet": "The OMA_AUTH_KEY environment variable isn’t set. LWCs won't be deployed.",
+  "manualDeploymentSteps": "<a href='%s' target='_blank'>Please refer to this documentation for manual deployment process</a>",
+  "errorRetrievingMetadata": "Error retrieving metadata",
+  "deploymentConsentNotGiven": "Deployment consent not given, manual deployment is required",
   "experienceSiteException": "We’ve encountered an exception while processing Experience Cloud sites.",
   "reservedKeysFoundInPropertySet": "Reserved keys found in any of output response transformation fields: %s.",
   "incompleteMigrationDetected": "We couldn't complete the migration process",
   "errorComponentMapping": "Error during component pre-processing: %s",
   "startingComponentPreProcessing": "Pre-processing components for name mapping",
   "completeComponentMappingMessage": "Registered name mappings for %s components",
   "componentMappingNotFound": "No registry mapping found for %s component: %s, using fallback cleaning",
-  "flexCardWithAngularOmniScriptWarning": "FlexCard has dependencies on Angular Omniscript(s) which are not migrated. Please convert OmniScript(s) to LWC before migrating this FlexCard.",
-  "angularOmniScriptDependencyWarning": "Element '%s' references Angular Omniscript '%s' which will not be migrated. Consider converting the referenced OmniScript to LWC",
+  "flexCardWithAngularOmniScriptWarning": "FlexCard has dependencies on Angular Omniscript(s) which are not migrated. Please convert Omniscript(s) to LWC before migrating this FlexCard.",
+  "angularOmniScriptDependencyWarning": "Element '%s' references Angular Omniscript '%s' which will not be migrated. Consider converting the referenced Omniscript to LWC",
   "skipFlexcardAngularOmniScriptDependencyWarning": "Skipping FlexCard %s due to Angular Omniscript dependencies",
   "flexCardMigrationProcessingMessage": "Processing %s FlexCards for migration (%s skipped due to Angular dependencies)",
   "noMetadataToDeploy": "No metadata to deploy",
@@ -235,6 +238,8 @@
   "skipStandardRuntimeDueToFailure": "Skipping Standard Runtime enablement due to designer setup failure",
   "errorCheckingStandardDesigner": "Failed to check standard designer status for package %s: %s",
   "relatedObjectsConsentNotGiven": "The related objects will not be processed because the user declined consent.",
+  "fileUpdatedToAllowRemoteCalls": "File has been updated to allow remote calls",
+  "fileUpdatedToAllowCalls": "File has been updated to allow calls",
   "startingCustomLabelMigration": "Starting Custom Labels migration",
   "customLabelMigrationCompleted": "Custom Labels migration completed. %s labels processed out of %s total",
   "errorDuringCustomLabelMigration": "Error during Custom Labels migration: %s",
@@ -254,4 +259,4 @@
   "errorFetchingCustomLabels": "Error fetching custom labels: %s",
   "customLabelAssessmentSummary": "Custom Label with same name and different value is already exist without namespace.",
   "generatedCustomLabelAssessmentReportPage": "Generated custom label assessment report page %s of %s with %s labels"
-}
+}

src/commands/omnistudio/migration/migrate.ts

@@ -674,7 +674,7 @@ export default class Migrate extends OmniStudioBaseCommand {
         } else if (!recordResults || recordResults.hasErrors) {
           obj.status = 'Failed';
         } else {
-          obj.status = 'Complete';
+          obj.status = 'Successfully migrated';
         }
 
         obj.errors = errors;

src/migration/customLabels.ts

@@ -1,3 +1,7 @@
+/* eslint-disable @typescript-eslint/no-explicit-any */
+/* eslint-disable @typescript-eslint/no-unsafe-assignment */
+/* eslint-disable @typescript-eslint/no-unsafe-member-access */
+/* eslint-disable @typescript-eslint/no-unsafe-call */
 import { UX } from '@salesforce/command';
 import { Connection, Messages } from '@salesforce/core';
 import { NetUtils, RequestMethod } from '../utils/net';

src/migration/dataraptor.ts

@@ -158,7 +158,7 @@ export class DataRaptorMigrationTool extends BaseMigrationTool implements Migrat
 
       // Verify duplicated names before trying to submitt
       if (duplicatedNames.has(transformedDataRaptor['Name'])) {
-        this.setRecordErrors(dr, this.messages.getMessage('duplicatedDrName'));
+        this.setRecordErrors(dr, this.messages.getMessage('duplicatedDrName', [transformedDataRaptor['Name']]));
         originalDrRecords.set(recordId, dr);
         continue;
       }
@@ -179,10 +179,7 @@ export class DataRaptorMigrationTool extends BaseMigrationTool implements Migrat
       if (drUploadResponse && drUploadResponse.success === true) {
         const items = await this.getItemsForDataRaptor(dataRaptorItemsData, name, drUploadResponse.id);
 
-        // Check for name changes
-        if (transformedDataRaptor[DRBundleMappings.Name] !== dr['Name']) {
-          drUploadResponse.newName = transformedDataRaptor[DRBundleMappings.Name];
-        }
+        drUploadResponse.newName = transformedDataRaptor[DRBundleMappings.Name];
 
         // Move the items
         await this.uploadTransformedData(DataRaptorMigrationTool.OMNIDATATRANSFORMITEM_NAME, items);
@@ -193,7 +190,7 @@ export class DataRaptorMigrationTool extends BaseMigrationTool implements Migrat
     progressBar.stop();
 
     return {
-      name: 'Data Mapper',
+      name: 'DataMappers',
       results: drUploadInfo,
       records: originalDrRecords,
     };