feat: move employee settings to export settings (#1115)

* feat: move employee settings to export settings

* qbo export settings

* lint fix

* html

* removed employee settings component

* fix lint

* fix tests

* fix onboarding

* lint fix

* pr comments

* lint fix

* pr comments

* pr comment / 2

* pr comments

* lint fix

* pr comment

Author: anishfyle

src/app/core/models/common/advanced-settings.model.ts

@@ -94,7 +94,7 @@ export class AdvancedSettingsModel {
       if (cccExportType && ['netsuite', 'quickbooks online', 'sage intacct'].includes(appName.toLowerCase()) && brandingConfig.brandId === 'fyle') {
         return defaultOptions;
       }
-      return defaultOptions.filter(option => option !== 'card_number');
+    return defaultOptions.filter(option => option !== 'card_number');
 
   }
 

src/app/core/models/enum/enum.model.ts

@@ -184,7 +184,6 @@ export enum IntacctOnboardingState {
 
 export enum QBOOnboardingState {
   CONNECTION = 'CONNECTION',
-  MAP_EMPLOYEES = 'MAP_EMPLOYEES',
   EXPORT_SETTINGS = 'EXPORT_SETTINGS',
   IMPORT_SETTINGS = 'IMPORT_SETTINGS',
   ADVANCED_CONFIGURATION = 'ADVANCED_CONFIGURATION',

src/app/core/models/qbo/qbo-configuration/qbo-export-setting.model.ts

@@ -301,4 +301,12 @@ export class QBOExportSettingModel extends ExportSettingModel {
 
     return exportSettingPayload;
   }
+
+  static createEmployeeSettingsForm(existingEmployeeFieldMapping: EmployeeFieldMapping, autoMapEmployees: boolean): FormGroup {
+    return new FormGroup({
+      employeeMapping: new FormControl(existingEmployeeFieldMapping, Validators.required),
+      autoMapEmployee: new FormControl(autoMapEmployees),
+      searchOption: new FormControl('')
+    });
+  }
 }

src/app/core/models/qbo/qbo-configuration/qbo-onboarding.model.ts

@@ -4,7 +4,6 @@ import { OnboardingStepper } from "../../misc/onboarding-stepper.model";
 
 type QBOOnboardingStepperMap = {
     [QBOOnboardingState.CONNECTION]: number,
-    [QBOOnboardingState.MAP_EMPLOYEES]: number,
     [QBOOnboardingState.EXPORT_SETTINGS]: number,
     [QBOOnboardingState.IMPORT_SETTINGS]: number,
     [QBOOnboardingState.ADVANCED_CONFIGURATION]: number,
@@ -24,14 +23,6 @@ export class QBOOnboardingModel {
           route: '/integrations/qbo/onboarding/connector',
           styleClasses: ['step-name-connector--text']
         },
-        {
-            active: false,
-            completed: false,
-            step: brandingContent.configuration.employeeSetting.stepName,
-            icon: 'mapping-medium',
-            route: '/integrations/qbo/onboarding/employee_settings',
-            styleClasses: ['step-name-export--text']
-        },
         {
           active: false,
           completed: false,
@@ -60,12 +51,11 @@ export class QBOOnboardingModel {
 
     private readonly onboardingStateStepMap: QBOOnboardingStepperMap = {
         [QBOOnboardingState.CONNECTION]: 1,
-        [QBOOnboardingState.MAP_EMPLOYEES]: 2,
-        [QBOOnboardingState.EXPORT_SETTINGS]: 3,
-        [QBOOnboardingState.IMPORT_SETTINGS]: 4,
-        [QBOOnboardingState.ADVANCED_CONFIGURATION]: 5,
-        [QBOOnboardingState.COMPLETE]: 6,
-        [QBOOnboardingState.CLONE_SETTINGS]: 7
+        [QBOOnboardingState.EXPORT_SETTINGS]: 2,
+        [QBOOnboardingState.IMPORT_SETTINGS]: 3,
+        [QBOOnboardingState.ADVANCED_CONFIGURATION]: 4,
+        [QBOOnboardingState.COMPLETE]: 5,
+        [QBOOnboardingState.CLONE_SETTINGS]: 6
       };
 
     getOnboardingSteps(currentStep: string, onboardingState: QBOOnboardingState): OnboardingStepper[] {

src/app/integrations/qbo/qbo-main/qbo-configuration/qbo-configuration-routing.module.ts

@@ -1,7 +1,6 @@
 import { NgModule } from '@angular/core';
 import { RouterModule, Routes } from '@angular/router';
 import { QboConfigurationComponent } from './qbo-configuration.component';
-import { QboEmployeeSettingsComponent } from '../../qbo-shared/qbo-employee-settings/qbo-employee-settings.component';
 import { QboExportSettingsComponent } from '../../qbo-shared/qbo-export-settings/qbo-export-settings.component';
 import { QboImportSettingsComponent } from '../../qbo-shared/qbo-import-settings/qbo-import-settings.component';
 import { QboAdvancedSettingsComponent } from '../../qbo-shared/qbo-advanced-settings/qbo-advanced-settings.component';
@@ -11,10 +10,6 @@ const routes: Routes = [
     path: '',
     component: QboConfigurationComponent,
     children: [
-      {
-        path: 'employee_settings',
-        component: QboEmployeeSettingsComponent
-      },
       {
         path: 'export_settings',
         component: QboExportSettingsComponent

src/app/integrations/qbo/qbo-main/qbo-configuration/qbo-configuration.component.ts

@@ -12,7 +12,6 @@ export class QboConfigurationComponent implements OnInit {
   readonly brandingContent = brandingContent.configuration;
 
   modules: MenuItem[] = [
-    {label: 'Map Employees', routerLink: '/integrations/qbo/main/configuration/employee_settings'},
     {label: this.brandingContent.exportSetting.stepName, routerLink: '/integrations/qbo/main/configuration/export_settings'},
     {label: this.brandingContent.importSetting.stepName, routerLink: '/integrations/qbo/main/configuration/import_settings'},
     {label: this.brandingContent.advancedSettings.stepName, routerLink: '/integrations/qbo/main/configuration/advanced_settings'}

src/app/integrations/qbo/qbo-onboarding/qbo-clone-settings/qbo-clone-settings.component.ts

@@ -218,7 +218,7 @@ export class QboCloneSettingsComponent implements OnInit {
   acceptWarning(data: ConfigurationWarningOut): void {
     this.isWarningDialogVisible = false;
     if (data.hasAccepted) {
-      this.router.navigate([`/integrations/qbo/onboarding/employee_settings`]);
+      this.router.navigate([`/integrations/qbo/onboarding/export_settings`]);
     }
   }
 

src/app/integrations/qbo/qbo-onboarding/qbo-onboarding-connector/qbo-onboarding-connector.component.ts

@@ -104,7 +104,7 @@ export class QboOnboardingConnectorComponent implements OnInit, OnDestroy {
     if (this.isContinueDisabled) {
       return;
     } else if (this.isCloneSettingsDisabled) {
-      this.router.navigate(['/integrations/qbo/onboarding/employee_settings']);
+      this.router.navigate(['/integrations/qbo/onboarding/export_settings']);
       return;
     }
 
@@ -145,7 +145,7 @@ export class QboOnboardingConnectorComponent implements OnInit, OnDestroy {
         this.isContinueDisabled = false;
         this.isCloneSettingsDisabled = true;
       } else {
-        this.router.navigate(['/integrations/qbo/onboarding/employee_settings']);
+        this.router.navigate(['/integrations/qbo/onboarding/export_settings']);
       }
     });
   }
@@ -167,11 +167,7 @@ export class QboOnboardingConnectorComponent implements OnInit, OnDestroy {
   }
 
   private handlePostQBOConnection(qboCredential: QBOCredential): void {
-    if (brandingFeatureConfig.featureFlags.mapEmployees) {
-      this.workspaceService.setOnboardingState(QBOOnboardingState.MAP_EMPLOYEES);
-    } else {
-      this.workspaceService.setOnboardingState(QBOOnboardingState.EXPORT_SETTINGS);
-    }
+    this.workspaceService.setOnboardingState(QBOOnboardingState.EXPORT_SETTINGS);
 
     this.qboConnectionInProgress = false;
     this.qboCompanyName = qboCredential.company_name;