fix: when redirected to reports page, it is coming up twice (#3120)

fylein · Jun 28, 2024 · 51a50bb · 51a50bb
1 parent fa61bc5
commit 51a50bb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/src/app/fyle/my-expenses/my-expenses.page.spec.ts b/src/app/fyle/my-expenses/my-expenses.page.spec.ts
@@ -3283,6 +3283,7 @@ describe('MyExpensesPage', () => {
       key: 'OPT_IN_POPUP_SHOWN_COUNT',
     });
     expect(component.showPromoteOptInModal).toHaveBeenCalledTimes(1);
+    expect(utilityService.toggleShowOptInAfterExpenseCreation).toHaveBeenCalledOnceWith(false);
   }));
 
   it('setModalDelay(): should set optInShowTimer and call showPromoteOptInModal after 2 seconds', fakeAsync(() => {

diff --git a/src/app/fyle/my-expenses/my-expenses.page.ts b/src/app/fyle/my-expenses/my-expenses.page.ts
@@ -806,6 +806,7 @@ export class MyExpensesPage implements OnInit {
 
           if (isAttemptLeft && isRedirectedFromAddExpense && canShowOptInModal) {
             this.showPromoteOptInModal();
+            this.utilityService.toggleShowOptInAfterExpenseCreation(false);
           }
         });
       }
-Original file line number
+Diff line change
@@ Expand Up / @@ -806,6 +806,7 @@ export class MyExpensesPage implements OnInit { @@
               if (isAttemptLeft && isRedirectedFromAddExpense && canShowOptInModal) {
                 this.showPromoteOptInModal();
+                this.utilityService.toggleShowOptInAfterExpenseCreation(false);
               }
             });
           }
@@ Expand Down @@