fix: address gemini review feedback for draw handler and import constants

Author: jeremi

spp_gis/static/src/js/widgets/gis_edit_map/field_gis_edit_map.esm.js

@@ -175,10 +175,18 @@ export class FieldGisEditMap extends Component {
 
         this._drawHandlers = {
             update(e) {
-                var data = self.draw.getAll();
-                if (data.features.length > 0) {
+                const eventFeature = e?.features?.[0];
+                if (eventFeature?.geometry) {
                     self.props.record.update({
-                        [self.props.name]: JSON.stringify(data.features[0].geometry),
+                        [self.props.name]: JSON.stringify(eventFeature.geometry),
+                    });
+                    return;
+                }
+
+                const allFeatures = self.draw.getAll()?.features || [];
+                if (allFeatures.length > 0 && allFeatures[0].geometry) {
+                    self.props.record.update({
+                        [self.props.name]: JSON.stringify(allFeatures[0].geometry),
                     });
                 }
             },

spp_program_geofence/models/eligibility_manager.py

@@ -38,6 +38,8 @@ class GeofenceMembershipManager(models.Model):
     _name = "spp.program.membership.manager.geofence"
     _inherit = ["spp.program.membership.manager", "spp.manager.source.mixin"]
     _description = "Geofence Eligibility Manager"
+    ASYNC_IMPORT_THRESHOLD = 1000
+    IMPORT_CHUNK_SIZE = 10000
 
     include_area_fallback = fields.Boolean(
         default=True,
@@ -187,7 +189,7 @@ def import_eligible_registrants(self, state="draft"):
         new_beneficiaries = new_beneficiaries.filtered(lambda r: r.id not in existing_partner_ids)
 
         ben_count = len(new_beneficiaries)
-        if ben_count < 1000:
+        if ben_count < self.ASYNC_IMPORT_THRESHOLD:
             self._import_registrants(new_beneficiaries, state=state, do_count=True)
         else:
             self._import_registrants_async(new_beneficiaries, state=state)
@@ -200,10 +202,10 @@ def _import_registrants_async(self, new_beneficiaries, state="draft"):
         program.write({"is_locked": True, "locked_reason": "Importing beneficiaries"})
 
         jobs = []
-        for i in range(0, len(new_beneficiaries), 10000):
+        for i in range(0, len(new_beneficiaries), self.IMPORT_CHUNK_SIZE):
             jobs.append(
                 self.delayable(channel="root_program.eligibility_manager")._import_registrants(
-                    new_beneficiaries[i : i + 10000], state
+                    new_beneficiaries[i : i + self.IMPORT_CHUNK_SIZE], state
                 )
             )
         main_job = group(*jobs)