feat: removed old my-expenses page (deprecation) - Part 2

src/app/core/mock-data/expense-filters.data.ts

@@ -1,4 +1,4 @@
-import { ExpenseFilters } from 'src/app/fyle/my-expenses-v2/my-expenses-filters.model';
+import { ExpenseFilters } from 'src/app/fyle/my-expenses/my-expenses-filters.model';
 import { DateFilters } from 'src/app/shared/components/fy-filters/date-filters.enum';
 import { ExpenseType } from '../enums/expense-type.enum';
 import { FilterState } from '../enums/filter-state.enum';

src/app/core/mock-data/modal-controller.data.ts

@@ -6,7 +6,7 @@ import { CreateNewReportComponent as createReportV2 } from 'src/app/shared/compo
 import { CreateNewReportComponent } from 'src/app/shared/components/create-new-report/create-new-report.component';
 import { Mode } from '@ionic/core';
 import { fyModalProperties } from './model-properties.data';
-import { AddTxnToReportDialogComponent as v2 } from 'src/app/fyle/my-expenses-v2/add-txn-to-report-dialog/add-txn-to-report-dialog.component';
+import { AddTxnToReportDialogComponent as v2 } from 'src/app/fyle/my-expenses/add-txn-to-report-dialog/add-txn-to-report-dialog.component';
 import { PopupAlertComponent } from 'src/app/shared/components/popup-alert/popup-alert.component';
 import { FilterOptions } from 'src/app/shared/components/fy-filters/filter-options.interface';
 import { DateFilters } from 'src/app/shared/components/fy-filters/date-filters.enum';

src/app/core/models/platform/expense-filters.model.ts

@@ -1,4 +1,4 @@
-import { Filters } from 'src/app/fyle/my-expenses-v2/my-expenses-filters.model';
+import { Filters } from 'src/app/fyle/my-expenses/my-expenses-filters.model';
 
 export interface ExpenseFilters extends Omit<Filters, 'state'> {
   state: string | string[];

src/app/core/services/platform/v1/shared/expenses.service.ts

@@ -9,7 +9,7 @@ import { PaymentModeSummary } from 'src/app/core/models/payment-mode-summary.mod
 import { AccountType } from 'src/app/core/models/platform/v1/account.model';
 import { Expense } from 'src/app/core/models/platform/v1/expense.model';
 import { GetExpenseQueryParam } from 'src/app/core/models/platform/v1/get-expenses-query.model';
-import { ExpenseFilters } from 'src/app/fyle/my-expenses-v2/my-expenses-filters.model';
+import { ExpenseFilters } from 'src/app/fyle/my-expenses/my-expenses-filters.model';
 import { DateFilters } from 'src/app/shared/components/fy-filters/date-filters.enum';
 import { DateService } from '../../../date.service';
 

src/app/core/services/tracking.service.ts

@@ -31,8 +31,8 @@ import {
   EnrollingNonRTFCardProperties,
 } from '../models/tracking-properties.model';
 import { ExpenseView } from '../models/expense-view.enum';
-import { ExpenseFilters } from 'src/app/fyle/my-expenses-v2/my-expenses-filters.model';
-import { ReportFilters } from 'src/app/fyle/my-expenses-v2/my-expenses-filters.model';
+import { ExpenseFilters } from 'src/app/fyle/my-expenses/my-expenses-filters.model';
+import { ReportFilters } from 'src/app/fyle/my-expenses/my-expenses-filters.model';
 import { TaskFilters } from '../models/task-filters.model';
 import { OrgCategory } from '../models/v1/org-category.model';
 import { TeamReportsFilters } from '../models/team-reports-filters.model';

src/app/core/services/transaction.service.ts

@@ -18,7 +18,7 @@ import { UserEventService } from './user-event.service';
 import { UndoMerge } from '../models/undo-merge.model';
 import { cloneDeep } from 'lodash';
 import { DateFilters } from 'src/app/shared/components/fy-filters/date-filters.enum';
-import { ExpenseFilters } from 'src/app/fyle/my-expenses-v2/my-expenses-filters.model';
+import { ExpenseFilters } from 'src/app/fyle/my-expenses/my-expenses-filters.model';
 import { PAGINATION_SIZE } from 'src/app/constants';
 import { PaymentModesService } from './payment-modes.service';
 import { OrgSettingsService } from './org-settings.service';

src/app/fyle/dashboard/stats/stats.component.spec.ts

@@ -356,7 +356,6 @@ describe('StatsComponent', () => {
 
   describe('goToExpensesPage():', () => {
     it('goToExpensesPage(): should navigate to expenses page with query params', () => {
-      component.redirectToNewPage$ = of(false);
       component.goToExpensesPage('COMPLETE');
 
       expect(router.navigate).toHaveBeenCalledOnceWith(['/', 'enterprise', 'my_expenses'], {
@@ -368,7 +367,6 @@ describe('StatsComponent', () => {
     });
 
     it('goToExpensesPage(): should navigate to expenses page with query params', () => {
-      component.redirectToNewPage$ = of(false);
       component.goToExpensesPage('INCOMPLETE');
 
       expect(router.navigate).toHaveBeenCalledOnceWith(['/', 'enterprise', 'my_expenses'], {
@@ -378,30 +376,6 @@ describe('StatsComponent', () => {
       });
       expect(trackingService.dashboardOnIncompleteExpensesClick).toHaveBeenCalledTimes(1);
     });
-
-    it('goToExpensesPage(): should navigate to v2 expenses page with query params', () => {
-      component.redirectToNewPage$ = of(true);
-      component.goToExpensesPage('COMPLETE');
-
-      expect(router.navigate).toHaveBeenCalledOnceWith(['/', 'enterprise', 'my_expenses_v2'], {
-        queryParams: {
-          filters: JSON.stringify({ state: ['READY_TO_REPORT'] }),
-        },
-      });
-      expect(trackingService.dashboardOnUnreportedExpensesClick).toHaveBeenCalledTimes(1);
-    });
-
-    it('goToExpensesPage(): should navigate to v2 expenses page with query params', () => {
-      component.redirectToNewPage$ = of(true);
-      component.goToExpensesPage('INCOMPLETE');
-
-      expect(router.navigate).toHaveBeenCalledOnceWith(['/', 'enterprise', 'my_expenses_v2'], {
-        queryParams: {
-          filters: JSON.stringify({ state: ['DRAFT'] }),
-        },
-      });
-      expect(trackingService.dashboardOnIncompleteExpensesClick).toHaveBeenCalledTimes(1);
-    });
   });
 
   describe('trackDashboardLaunchTime():', () => {

src/app/fyle/dashboard/stats/stats.component.ts

@@ -200,23 +200,20 @@ export class StatsComponent implements OnInit {
   }
 
   goToExpensesPage(state: string): void {
-    this.redirectToNewPage$.subscribe((redirect) => {
-      const endpoint = redirect ? 'my_expenses_v2' : 'my_expenses';
-      if (state === 'COMPLETE') {
-        const queryParams: Params = { filters: JSON.stringify({ state: ['READY_TO_REPORT'] }) };
-        this.router.navigate(['/', 'enterprise', endpoint], {
-          queryParams,
-        });
+    if (state === 'COMPLETE') {
+      const queryParams: Params = { filters: JSON.stringify({ state: ['READY_TO_REPORT'] }) };
+      this.router.navigate(['/', 'enterprise', 'my_expenses'], {
+        queryParams,
+      });
 
-        this.trackingService.dashboardOnUnreportedExpensesClick();
-      } else {
-        const queryParams: Params = { filters: JSON.stringify({ state: ['DRAFT'] }) };
-        this.router.navigate(['/', 'enterprise', endpoint], {
-          queryParams,
-        });
-        this.trackingService.dashboardOnIncompleteExpensesClick();
-      }
-    });
+      this.trackingService.dashboardOnUnreportedExpensesClick();
+    } else {
+      const queryParams: Params = { filters: JSON.stringify({ state: ['DRAFT'] }) };
+      this.router.navigate(['/', 'enterprise', 'my_expenses'], {
+        queryParams,
+      });
+      this.trackingService.dashboardOnIncompleteExpensesClick();
+    }
   }
 
   private trackDashboardLaunchTime(): void {

src/app/fyle/dashboard/tasks/tasks-3.component.spec.ts

@@ -19,7 +19,7 @@ import { cloneDeep, noop } from 'lodash';
 import { snackbarPropertiesRes2 } from 'src/app/core/mock-data/snackbar-properties.data';
 import { ToastMessageComponent } from 'src/app/shared/components/toast-message/toast-message.component';
 import { ToastType } from 'src/app/core/enums/toast-type.enum';
-import { AddTxnToReportDialogComponent } from '../../my-expenses-v2/add-txn-to-report-dialog/add-txn-to-report-dialog.component';
+import { AddTxnToReportDialogComponent } from '../../my-expenses/add-txn-to-report-dialog/add-txn-to-report-dialog.component';
 import { ExpensesService } from 'src/app/core/services/platform/v1/spender/expenses.service';
 import { expenseData } from 'src/app/core/mock-data/platform/v1/expense.data';
 import { unreportedExpensesQueryParams } from 'src/app/core/mock-data/platform/v1/expenses-query-params.data';

src/app/fyle/dashboard/tasks/tasks.component.ts

@@ -22,7 +22,7 @@ import { FilterOptionType } from 'src/app/shared/components/fy-filters/filter-op
 import { FilterOptions } from 'src/app/shared/components/fy-filters/filter-options.interface';
 import { FyFiltersComponent } from 'src/app/shared/components/fy-filters/fy-filters.component';
 import { ToastMessageComponent } from 'src/app/shared/components/toast-message/toast-message.component';
-import { AddTxnToReportDialogComponent } from '../../my-expenses-v2/add-txn-to-report-dialog/add-txn-to-report-dialog.component';
+import { AddTxnToReportDialogComponent } from '../../my-expenses/add-txn-to-report-dialog/add-txn-to-report-dialog.component';
 import { FilterPill } from 'src/app/shared/components/fy-filter-pills/filter-pill.interface';
 import { SelectedFilters } from 'src/app/shared/components/fy-filters/selected-filters.interface';
 import { ExpensesService } from 'src/app/core/services/platform/v1/spender/expenses.service';

src/app/fyle/fyle-routing.module.ts

@@ -1,16 +1,14 @@
 import { NgModule } from '@angular/core';
 import { Routes, RouterModule } from '@angular/router';
-import { MyExpensesGuardGuard } from '../core/guards/my-expenses-guard.guard';
-import { BetaPageFeatureFlagGuard } from '../core/guards/beta-page-feature-flag.guard';
 
 const routes: Routes = [
   {
     path: 'my_dashboard',
     loadChildren: () => import('./dashboard/dashboard.module').then((m) => m.DashboardPageModule),
   },
   {
-    path: 'my_expenses_v2',
-    loadChildren: () => import('./my-expenses-v2/my-expenses-v2.module').then((m) => m.MyExpensesV2PageModule),
+    path: 'my_expenses',
+    loadChildren: () => import('./my-expenses/my-expenses.module').then((m) => m.MyExpensesPageModule),
   },
   {
     path: 'my_advances',

src/app/fyle/my-expenses-v2/my-expenses-routing.module.ts → src/app/fyle/my-expenses/my-expenses-routing.module.ts

@@ -1,17 +1,17 @@
 import { NgModule } from '@angular/core';
 import { Routes, RouterModule } from '@angular/router';
 
-import { MyExpensesV2Page } from './my-expenses-v2.page';
+import { MyExpensesPage } from './my-expenses.page';
 
 const routes: Routes = [
   {
     path: '',
-    component: MyExpensesV2Page,
+    component: MyExpensesPage,
   },
 ];
 
 @NgModule({
   imports: [RouterModule.forChild(routes)],
   exports: [RouterModule],
 })
-export class MyExpensesV2PageRoutingModule {}
+export class MyExpensesPageRoutingModule {}

src/app/fyle/my-expenses-v2/my-expenses-v2.module.ts → src/app/fyle/my-expenses/my-expenses.module.ts

@@ -2,8 +2,8 @@ import { NgModule } from '@angular/core';
 import { CommonModule } from '@angular/common';
 import { FormsModule, ReactiveFormsModule } from '@angular/forms';
 import { IonicModule } from '@ionic/angular';
-import { MyExpensesV2PageRoutingModule } from './my-expenses-routing.module';
-import { MyExpensesV2Page } from './my-expenses-v2.page';
+import { MyExpensesPageRoutingModule } from './my-expenses-routing.module';
+import { MyExpensesPage } from './my-expenses.page';
 import { MatFormFieldModule } from '@angular/material/form-field';
 import { MatInputModule } from '@angular/material/input';
 import { MatButtonModule } from '@angular/material/button';
@@ -22,7 +22,7 @@ import { MatCheckboxModule } from '@angular/material/checkbox';
     CommonModule,
     FormsModule,
     IonicModule,
-    MyExpensesV2PageRoutingModule,
+    MyExpensesPageRoutingModule,
     MatInputModule,
     MatFormFieldModule,
     MatButtonModule,
@@ -40,6 +40,6 @@ import { MatCheckboxModule } from '@angular/material/checkbox';
     SharedModule,
     MatCheckboxModule,
   ],
-  declarations: [MyExpensesV2Page, AddTxnToReportDialogComponent],
+  declarations: [MyExpensesPage, AddTxnToReportDialogComponent],
 })
-export class MyExpensesV2PageModule {}
+export class MyExpensesPageModule {}

src/app/fyle/my-expenses-v2/my-expenses-v2.page.spec.ts → src/app/fyle/my-expenses/my-expenses.page.spec.ts

@@ -121,13 +121,13 @@ import { MaskNumber } from 'src/app/shared/pipes/mask-number.pipe';
 import { ReportState } from 'src/app/shared/pipes/report-state.pipe';
 import { environment } from 'src/environments/environment';
 import { AddTxnToReportDialogComponent } from './add-txn-to-report-dialog/add-txn-to-report-dialog.component';
-import { MyExpensesV2Page } from './my-expenses-v2.page';
+import { MyExpensesPage } from './my-expenses.page';
 import { MyExpensesService } from './my-expenses.service';
 import { completeStats, incompleteStats } from 'src/app/core/mock-data/platform/v1/expenses-stats.data';
 
 describe('MyExpensesV2Page', () => {
-  let component: MyExpensesV2Page;
-  let fixture: ComponentFixture<MyExpensesV2Page>;
+  let component: MyExpensesPage;
+  let fixture: ComponentFixture<MyExpensesPage>;
   let tasksService: jasmine.SpyObj<TasksService>;
   let currencyService: jasmine.SpyObj<CurrencyService>;
   let reportService: jasmine.SpyObj<ReportService>;
@@ -286,7 +286,7 @@ describe('MyExpensesV2Page', () => {
     ]);
 
     TestBed.configureTestingModule({
-      declarations: [MyExpensesV2Page, ReportState, MaskNumber],
+      declarations: [MyExpensesPage, ReportState, MaskNumber],
       imports: [IonicModule.forRoot(), RouterTestingModule, HttpClientTestingModule],
       providers: [
         { provide: TasksService, useValue: tasksServiceSpy },
@@ -395,7 +395,7 @@ describe('MyExpensesV2Page', () => {
       schemas: [NO_ERRORS_SCHEMA],
     }).compileComponents();
 
-    fixture = TestBed.createComponent(MyExpensesV2Page);
+    fixture = TestBed.createComponent(MyExpensesPage);
     component = fixture.componentInstance;
 
     activatedRoute = TestBed.inject(ActivatedRoute) as jasmine.SpyObj<ActivatedRoute>;

src/app/fyle/my-expenses-v2/my-expenses-v2.page.ts → src/app/fyle/my-expenses/my-expenses.page.ts

@@ -80,10 +80,10 @@ import { MyExpensesService } from './my-expenses.service';
 
 @Component({
   selector: 'app-my-expenses',
-  templateUrl: './my-expenses-v2.page.html',
-  styleUrls: ['./my-expenses-v2.page.scss'],
+  templateUrl: './my-expenses.page.html',
+  styleUrls: ['./my-expenses.page.scss'],
 })
-export class MyExpensesV2Page implements OnInit {
+export class MyExpensesPage implements OnInit {
   @ViewChild('simpleSearchInput') simpleSearchInput: ElementRef<HTMLInputElement>;
 
   isConnected$: Observable<boolean>;

src/app/fyle/my-reports/my-reports.page.spec.ts

@@ -31,7 +31,7 @@ import { LoaderService } from 'src/app/core/services/loader.service';
 import { TrackingService } from 'src/app/core/services/tracking.service';
 import { SelectedFilters } from 'src/app/shared/components/fy-filters/selected-filters.interface';
 import { FilterPill } from 'src/app/shared/components/fy-filter-pills/filter-pill.interface';
-import { Filters } from '../my-expenses-v2/my-expenses-filters.model';
+import { Filters } from '../my-expenses/my-expenses-filters.model';
 import {
   selectedFilters1,
   selectedFilters2,

src/app/shared/components/spent-cards/card-detail/card-detail.component.spec.ts

@@ -101,7 +101,7 @@ describe('CardDetailComponent', () => {
 
       component.goToExpensesPage('incompleteExpenses', component.cardDetail);
       expect(trackingService.dashboardOnIncompleteCardExpensesClick).toHaveBeenCalledTimes(1);
-      expect(router.navigate).toHaveBeenCalledOnceWith(['/', 'enterprise', 'my_expenses_v2'], {
+      expect(router.navigate).toHaveBeenCalledOnceWith(['/', 'enterprise', 'my_expenses'], {
         queryParams,
       });
     });
@@ -119,7 +119,7 @@ describe('CardDetailComponent', () => {
 
       component.goToExpensesPage('totalExpenses', component.cardDetail);
       expect(trackingService.dashboardOnTotalCardExpensesClick).toHaveBeenCalledTimes(1);
-      expect(router.navigate).toHaveBeenCalledOnceWith(['/', 'enterprise', 'my_expenses_v2'], {
+      expect(router.navigate).toHaveBeenCalledOnceWith(['/', 'enterprise', 'my_expenses'], {
         queryParams,
       });
     });

src/app/shared/components/spent-cards/card-detail/card-detail.component.ts

@@ -1,7 +1,5 @@
-import { Component, Input, OnInit } from '@angular/core';
+import { Component, Input } from '@angular/core';
 import { Params, Router } from '@angular/router';
-import { map, noop } from 'rxjs';
-import { CardStatus } from 'src/app/core/enums/card-status.enum';
 import { PlatformCorporateCardDetail } from 'src/app/core/models/platform-corporate-card-detail.model';
 import { OrgSettingsService } from 'src/app/core/services/org-settings.service';
 import { TrackingService } from 'src/app/core/services/tracking.service';
@@ -11,41 +9,26 @@ import { TrackingService } from 'src/app/core/services/tracking.service';
   templateUrl: './card-detail.component.html',
   styleUrls: ['./card-detail.component.scss'],
 })
-export class CardDetailComponent implements OnInit {
+export class CardDetailComponent {
   @Input() cardDetail: PlatformCorporateCardDetail;
 
   @Input() homeCurrency: string;
 
   @Input() currencySymbol: string;
 
-  redirectToNewPage = false;
-
   constructor(
     private router: Router,
     private trackingService: TrackingService,
     private orgSettingService: OrgSettingsService
   ) {}
 
-  ngOnInit(): void {
-    this.orgSettingService
-      .get()
-      .pipe(
-        map((orgSettings) => {
-          if (orgSettings.mobile_app_my_expenses_beta_enabled) {
-            this.redirectToNewPage = true;
-          }
-        })
-      )
-      .subscribe(noop);
-  }
-
   goToExpensesPage(state: string, cardDetail: PlatformCorporateCardDetail): void {
     if (state === 'incompleteExpenses' && cardDetail.stats.totalDraftTxns && cardDetail.stats.totalDraftTxns > 0) {
       const queryParams: Params = {
         filters: JSON.stringify({ state: ['DRAFT'], cardNumbers: [this.cardDetail?.card.card_number] }),
       };
 
-      this.router.navigate(['/', 'enterprise', `${this.redirectToNewPage ? 'my_expenses_v2' : 'my_expenses'}`], {
+      this.router.navigate(['/', 'enterprise', 'my_expenses'], {
         queryParams,
       });
 
@@ -54,7 +37,7 @@ export class CardDetailComponent implements OnInit {
       const queryParams: Params = {
         filters: JSON.stringify({ state: ['DRAFT,READY_TO_REPORT'], cardNumbers: [this.cardDetail?.card.card_number] }),
       };
-      this.router.navigate(['/', 'enterprise', `${this.redirectToNewPage ? 'my_expenses_v2' : 'my_expenses'}`], {
+      this.router.navigate(['/', 'enterprise', 'my_expenses'], {
         queryParams,
       });