From 5f7de01ba9844b3ab24dd357adf7e206faa9848a Mon Sep 17 00:00:00 2001
From: Jaanu423 <puja.mourya575@gmail.com>
Date: Wed, 3 Jun 2026 13:02:06 +0530
Subject: [PATCH] fix(savings): remove duplicate Submit button in Modify
 Savings Account form
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Added hideButtons @Input to SavingsAccountPreviewStepComponent and passed
hideButtons=true from the edit form — prevents the preview step's own
Cancel/Submit row from rendering alongside the tab action bar's buttons.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../edit-savings-account.component.html       |  1 +
 ...avings-account-preview-step.component.html | 20 ++++++++++---------
 .../savings-account-preview-step.component.ts |  2 ++
 3 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/src/app/savings/edit-savings-account/edit-savings-account.component.html b/src/app/savings/edit-savings-account/edit-savings-account.component.html
index f262b50d0..35421b324 100644
--- a/src/app/savings/edit-savings-account/edit-savings-account.component.html
+++ b/src/app/savings/edit-savings-account/edit-savings-account.component.html
@@ -120,6 +120,7 @@
             [savingsAccount]="savingsAccount"
             [savingsAccountDigital]="savingsAccountDigital"
             [savingsAccountNominees]="savingsAccountCurrencyStep?.nominees ?? []"
+            [hideButtons]="true"
             (submitEvent)="submit()"
           >
           </mifosx-savings-account-preview-step>
diff --git a/src/app/savings/savings-account-stepper/savings-account-preview-step/savings-account-preview-step.component.html b/src/app/savings/savings-account-stepper/savings-account-preview-step/savings-account-preview-step.component.html
index c261103a1..40d77d84a 100644
--- a/src/app/savings/savings-account-stepper/savings-account-preview-step/savings-account-preview-step.component.html
+++ b/src/app/savings/savings-account-stepper/savings-account-preview-step/savings-account-preview-step.component.html
@@ -581,12 +581,14 @@
   mat-stepper. Re-introducing matStepperPrevious here would require a
   CdkStepper ancestor and would crash the view on init.
 -->
-<div class="preview-submit-row">
-  <button mat-stroked-button [routerLink]="['../..']">
-    {{ 'labels.buttons.Cancel' | translate }}
-  </button>
-  <button mat-raised-button color="primary" class="preview-submit-row__primary" (click)="submitEvent.emit()">
-    <fa-icon icon="check" class="m-r-10"></fa-icon>
-    {{ 'labels.buttons.Submit' | translate }}
-  </button>
-</div>
+@if (!hideButtons) {
+  <div class="preview-submit-row">
+    <button mat-stroked-button [routerLink]="['../..']">
+      {{ 'labels.buttons.Cancel' | translate }}
+    </button>
+    <button mat-raised-button color="primary" class="preview-submit-row__primary" (click)="submitEvent.emit()">
+      <fa-icon icon="check" class="m-r-10"></fa-icon>
+      {{ 'labels.buttons.Submit' | translate }}
+    </button>
+  </div>
+}
diff --git a/src/app/savings/savings-account-stepper/savings-account-preview-step/savings-account-preview-step.component.ts b/src/app/savings/savings-account-stepper/savings-account-preview-step/savings-account-preview-step.component.ts
index 8ca52d826..8f2cea945 100644
--- a/src/app/savings/savings-account-stepper/savings-account-preview-step/savings-account-preview-step.component.ts
+++ b/src/app/savings/savings-account-stepper/savings-account-preview-step/savings-account-preview-step.component.ts
@@ -92,6 +92,8 @@ export class SavingsAccountPreviewStepComponent implements OnChanges {
   @Input() savingsAccountNominees: any[] = [];
   /** active Client Members in case of GSIM Account */
   @Input() activeClientMembers?: any;
+  /** Hide internal Cancel/Submit buttons when parent provides its own action bar */
+  @Input() hideButtons = false;
 
   /** Table Data Source */
   dataSource: any;