Add allow_null_values to target column config

Author: utkarsh14

app/blueprints/docs/surveystream.yml

@@ -3687,9 +3687,9 @@ paths:
                         description: >-
                           The field type of the column. Can be one of
                           `personal_details`, `custom_field`, or `location`.
-                      bulk_editable:
+                      allow_null_values:
                         type: boolean
-                        description: Whether the field is bulk editable or not
+                        description: Whether the field allows null values or not
                       column_source:
                         type: string
                         description: >-
@@ -3698,7 +3698,6 @@ paths:
                     required:
                       - column_name
                       - column_type
-                      - bulk_editable
                       - column_source
               required:
                 - form_uid
@@ -3757,9 +3756,9 @@ paths:
                               description: >
                                 The field type of the column. Can be one of
                                 `personal_details`, `custom_field`, or `location`.
-                            bulk_editable:
+                            allow_null_values:
                               type: boolean
-                              description: Whether the field is bulk editable or not.
+                              description: Whether the field allows null values or not.
                             column_source:
                               type: string
                               description: >-
@@ -7382,13 +7381,12 @@ paths:
                         description: >-
                           The field type of the column. Can be one of
                           `basic_details`, `custom_field`, or `location`.
-                      bulk_editable:
+                      allow_null_values:
                         type: boolean
-                        description: Whether the field is bulk editable or not
+                        description: Whether the field allows null values or not
                     required:
                       - column_name
                       - column_type
-                      - bulk_editable
                 filters:
                   type: array
                   description: >-
@@ -7459,9 +7457,9 @@ paths:
                         items:
                           type: object
                           properties:
-                            bulk_editable:
+                            allow_null_values:
                               type: boolean
-                              description: Whether the field is bulk editable or not.
+                              description: Whether the field allows null values or not.
                             column_name:
                               type: string
                               description: The field name of the column.

app/blueprints/locations/controllers.py

@@ -1042,6 +1042,25 @@ def append_locations(validated_query_params, validated_payload):
             ),
             422,
         )
+    except Exception as e:
+        return (
+            jsonify(
+                {
+                    "success": False,
+                    "record_errors": {
+                        "summary_by_error_type": [
+                            {
+                                "error_type": "File processing error",
+                                "error_message": str(e),
+                                "error_count": 1,
+                                "row_numbers_with_errors": [],
+                            }
+                        ]
+                    },
+                }
+            ),
+            422,
+        )
 
     # Get the existing locations for the survey
 
@@ -1126,6 +1145,25 @@ def append_locations(validated_query_params, validated_payload):
             ),
             422,
         )
+    except Exception as e:
+        return (
+            jsonify(
+                {
+                    "success": False,
+                    "record_errors": {
+                        "summary_by_error_type": [
+                            {
+                                "error_type": "File processing error",
+                                "error_message": str(e),
+                                "error_count": 1,
+                                "row_numbers_with_errors": [],
+                            }
+                        ]
+                    },
+                }
+            ),
+            422,
+        )
 
     for i, geo_level in enumerate(geo_level_hierarchy.ordered_geo_levels):
         # Get the location_id and location_name column names for the geo level

app/blueprints/targets/controllers.py

@@ -1008,9 +1008,9 @@ def bulk_update_targets(validated_payload):
         "location": [],
         "custom_fields": [],
     }
-
     for column in column_config:
-        if column.bulk_editable:
+        # Skip target_id column as it shouldn't be bulk editable
+        if column.column_name != "target_id":
             bulk_editable_fields[column.column_type].append(column.column_name)
 
     for key in payload.keys():
@@ -1174,16 +1174,6 @@ def update_target_column_config(validated_payload):
     db.session.flush()
 
     for column in payload["column_config"]:
-        if not isinstance(column["bulk_editable"], bool):
-            return (
-                jsonify(
-                    {
-                        "success": False,
-                        "errors": f"Field 'bulk_editable' must be a boolean",
-                    }
-                ),
-                422,
-            )
         if not isinstance(column["contains_pii"], bool):
             return (
                 jsonify(
@@ -1200,7 +1190,7 @@ def update_target_column_config(validated_payload):
                 form_uid=form_uid,
                 column_name=column["column_name"],
                 column_type=column["column_type"],
-                bulk_editable=column["bulk_editable"],
+                allow_null_values=column["allow_null_values"],
                 contains_pii=column["contains_pii"],
                 column_source=column["column_source"],
             )
@@ -1254,7 +1244,7 @@ def get_target_column_config(validated_query_params):
         {
             "column_name": column.column_name,
             "column_type": column.column_type,
-            "bulk_editable": column.bulk_editable,
+            "allow_null_values": column.allow_null_values,
             "contains_pii": column.contains_pii,
             "column_source": column.column_source,
         }

app/blueprints/targets/models.py

@@ -152,7 +152,7 @@ class TargetColumnConfig(db.Model):
         ),
         nullable=False,
     )
-    bulk_editable = db.Column(db.Boolean(), nullable=False, default=False)
+    allow_null_values = db.Column(db.Boolean(), nullable=False, server_default="true")
     contains_pii = db.Column(db.Boolean(), nullable=True)
     column_source = db.Column(
         db.String(),

migrations/versions/b0c4574bd52e_.py

@@ -0,0 +1,43 @@
+"""
+Targets Column Config : Add allow_null_values column, remove bulk_editable column
+
+Revision ID: b0c4574bd52e
+Revises: c5530d333545
+Create Date: 2025-07-17 07:14:01.333110
+
+"""
+
+import sqlalchemy as sa
+from alembic import op
+
+# revision identifiers, used by Alembic.
+revision = "b0c4574bd52e"
+down_revision = "c5530d333545"
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    with op.batch_alter_table("target_column_config", schema="webapp") as batch_op:
+        batch_op.add_column(
+            sa.Column(
+                "allow_null_values", sa.Boolean(), server_default="true", nullable=False
+            )
+        )
+        batch_op.drop_column("bulk_editable")
+
+    # ### end Alembic commands ###
+
+
+def downgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    with op.batch_alter_table("target_column_config", schema="webapp") as batch_op:
+        batch_op.add_column(
+            sa.Column(
+                "bulk_editable", sa.BOOLEAN(), autoincrement=False, nullable=False
+            )
+        )
+        batch_op.drop_column("allow_null_values")
+
+    # ### end Alembic commands ###