From eb88fb0813d4d06c38fd12b0c71eee5d7d5fa5da Mon Sep 17 00:00:00 2001
From: Jaanu423 <puja.mourya575@gmail.com>
Date: Thu, 9 Jul 2026 15:47:14 +0530
Subject: [PATCH] fix(db): add missing m_savings_account columns causing 500 on
 savings list

introducer_required, share_linking_enabled, terminate_true were missing
from m_savings_account in fineract_saga. Applied directly to DB and
added 0368 catch-up migration with preConditions for idempotency.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../db/changelog/tenant/changelog-tenant.xml  |  1 +
 .../0368_catchup_savings_account_columns.xml  | 35 +++++++++++++++++++
 2 files changed, 36 insertions(+)
 create mode 100644 fineract-provider/src/main/resources/db/changelog/tenant/parts/0368_catchup_savings_account_columns.xml

diff --git a/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml b/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml
index 8629886a9..e51d9ed53 100644
--- a/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml
+++ b/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml
@@ -397,4 +397,5 @@
     <include file="parts/0338_add_soft_delete_to_tellers.xml" relativeToChangelogFile="true"/>
     <include file="parts/0339_add_share_linking_enabled_to_current_account.xml" relativeToChangelogFile="true"/>
     <include file="parts/0340_add_terminate_true_to_savings_and_current_account.xml" relativeToChangelogFile="true"/>
+    <include file="parts/0368_catchup_savings_account_columns.xml" relativeToChangelogFile="true"/>
 </databaseChangeLog>
diff --git a/fineract-provider/src/main/resources/db/changelog/tenant/parts/0368_catchup_savings_account_columns.xml b/fineract-provider/src/main/resources/db/changelog/tenant/parts/0368_catchup_savings_account_columns.xml
new file mode 100644
index 000000000..a7c83c5bb
--- /dev/null
+++ b/fineract-provider/src/main/resources/db/changelog/tenant/parts/0368_catchup_savings_account_columns.xml
@@ -0,0 +1,35 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<databaseChangeLog xmlns="http://www.liquibase.org/xml/ns/dbchangelog"
+                   xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+                   xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-4.1.xsd">
+
+    <!-- Catch-up: columns added to m_savings_account that were missing in fineract_saga -->
+
+    <changeSet id="0368-sa-introducer-required" author="CBS">
+        <preConditions onFail="MARK_RAN">
+            <not><columnExists tableName="m_savings_account" columnName="introducer_required"/></not>
+        </preConditions>
+        <addColumn tableName="m_savings_account">
+            <column name="introducer_required" type="BOOLEAN" defaultValueBoolean="false"/>
+        </addColumn>
+    </changeSet>
+
+    <changeSet id="0368-sa-share-linking-enabled" author="CBS">
+        <preConditions onFail="MARK_RAN">
+            <not><columnExists tableName="m_savings_account" columnName="share_linking_enabled"/></not>
+        </preConditions>
+        <addColumn tableName="m_savings_account">
+            <column name="share_linking_enabled" type="BOOLEAN" defaultValueBoolean="false"/>
+        </addColumn>
+    </changeSet>
+
+    <changeSet id="0368-sa-terminate-true" author="CBS">
+        <preConditions onFail="MARK_RAN">
+            <not><columnExists tableName="m_savings_account" columnName="terminate_true"/></not>
+        </preConditions>
+        <addColumn tableName="m_savings_account">
+            <column name="terminate_true" type="BOOLEAN" defaultValueBoolean="false"/>
+        </addColumn>
+    </changeSet>
+
+</databaseChangeLog>