Skip to content

Commit 94da9fb

Browse files
committed
test and datafixes
1 parent 9491a8e commit 94da9fb

File tree

9 files changed

+8
-19
lines changed

9 files changed

+8
-19
lines changed

src/app/fyle/add-edit-per-diem/add-edit-per-diem.page.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -892,10 +892,6 @@ export class AddEditPerDiemPage implements OnInit {
892892
map((orgSettings) => orgSettings.advanced_per_diems_settings?.enable_employee_restriction)
893893
);
894894

895-
orgSettings$.subscribe((orgSettings) => {
896-
897-
});
898-
899895
this.setupNetworkWatcher();
900896

901897
this.recentlyUsedValues$ = this.isConnected$.pipe(

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,7 @@ describe('StatsComponent', () => {
183183
expect(dashboardService.getReportsStats).toHaveBeenCalledTimes(1);
184184
expect(orgSettingsService.get).toHaveBeenCalledTimes(1);
185185
expect(paymentModeService.isNonReimbursableOrg).toHaveBeenCalledOnceWith(
186-
orgSettingsParamsWithSimplifiedReport.payment_mode_settings
186+
orgSettingsRes.payment_mode_settings
187187
);
188188
expect(component.reportStatsLoading).toBeFalse();
189189
done();

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -495,7 +495,6 @@ export class MyExpensesPage implements OnInit {
495495
this.isPerDiemEnabled$ = this.orgSettings$.pipe(map((orgSettings) => orgSettings?.per_diem?.enabled));
496496

497497
this.orgSettings$.subscribe((orgSettings) => {
498-
499498
this.restrictPendingTransactionsEnabled =
500499
(orgSettings?.corporate_credit_card_settings?.enabled &&
501500
orgSettings.pending_cct_expense_restriction?.enabled) ||

src/app/fyle/my-view-report/my-view-report.page.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ import { Expense } from 'src/app/core/models/platform/v1/expense.model';
2323
import { ExpenseView } from 'src/app/core/models/expense-view.enum';
2424
import { OrgSettingsService } from 'src/app/core/services/org-settings.service';
2525
import { ReportPageSegment } from 'src/app/core/enums/report-page-segment.enum';
26-
import { OrgSettings } from 'src/app/core/models/org-settings.model';
2726
import { ExtendedOrgUser } from 'src/app/core/models/extended-org-user.model';
2827
import { ExpensesService } from 'src/app/core/services/platform/v1/spender/expenses.service';
2928
import { AddExpensesToReportComponent } from './add-expenses-to-report/add-expenses-to-report.component';

src/app/fyle/team-reports/team-reports.page.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -130,9 +130,6 @@ export class TeamReportsPage implements OnInit {
130130
this.teamReportsTaskCount = teamReportsTaskCount;
131131
});
132132

133-
const orgSettings$ = this.orgSettingsService.get().pipe(shareReplay(1));
134-
135-
136133
this.eou$.subscribe((eou: ExtendedOrgUser) => {
137134
this.loadData$ = new BehaviorSubject({
138135
pageNumber: 1,

src/app/fyle/view-expense/view-expense.page.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,6 @@ export class ViewExpensePage {
395395
this.isRTFEnabled =
396396
(orgSettings.visa_enrollment_settings.allowed && orgSettings.visa_enrollment_settings.enabled) ||
397397
(orgSettings.mastercard_enrollment_settings.allowed && orgSettings.mastercard_enrollment_settings.enabled);
398-
399398
});
400399

401400
this.expenseFieldsService

src/app/fyle/view-mileage/view-mileage.page.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -374,7 +374,6 @@ export class ViewMileagePage {
374374
.pipe(shareReplay(1))
375375
.subscribe((orgSettings) => {
376376
this.orgSettings = orgSettings;
377-
378377
});
379378

380379
this.mileageCustomFields$ = this.mileageExpense$.pipe(

src/app/shared/components/fy-add-to-report/fy-add-to-report-modal/fy-add-to-report-modal.component.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ export class FyAddToReportModalComponent implements OnInit, AfterViewInit {
5151
private currencyService: CurrencyService
5252
) {}
5353

54-
ngOnInit() {
54+
ngOnInit(): void {
5555
if (this.currentSelection) {
5656
this.options = this.options
5757
.map((option) =>
@@ -65,23 +65,23 @@ export class FyAddToReportModalComponent implements OnInit, AfterViewInit {
6565
});
6666
}
6767

68-
ngAfterViewInit() {
68+
ngAfterViewInit(): void {
6969
this.cdr.detectChanges();
7070
}
7171

72-
onDoneClick() {
72+
onDoneClick(): void {
7373
this.modalController.dismiss();
7474
}
7575

76-
onElementSelect(option: Option) {
76+
onElementSelect(option: Option): void {
7777
this.modalController.dismiss(option);
7878
}
7979

80-
createDraftReport() {
80+
createDraftReport(): void {
8181
this.modalController.dismiss({ createDraftReport: true });
8282
}
8383

84-
dismissModal(event: { srcElement: { innerText: string } }) {
84+
dismissModal(event: { srcElement: { innerText: string } }): void {
8585
this.modalController.dismiss({
8686
label: event.srcElement.innerText,
8787
value: null,

src/app/shared/components/fy-filters/fy-filters.component.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { FilterOptions } from './filter-options.interface';
33
import { SelectedFilters } from './selected-filters.interface';
44
import { FilterOptionType } from './filter-option-type.enum';
55
import { ModalController } from '@ionic/angular';
6-
import { forkJoin, Observable, of } from 'rxjs';
6+
import { Observable, of } from 'rxjs';
77

88
@Component({
99
selector: 'app-fy-filters',

0 commit comments

Comments
 (0)