[ENG-11762] Refactor filter handling in notification campaign (#11813)

Author: Ostap-Zherebetskyi

admin/notifications/views.py

@@ -561,10 +561,11 @@ def get_queryset(self):
             if predefined := json_filters.get('predefined'):
                 filters = FILTER_PRESETS.get(predefined, {})
             else:
-                filters = {
-                    f'{item["field"]}__{item["lookup"]}': item['value']
-                    for item in json_filters.get('manual', [])
-                }
+                for item in json_filters.get('manual', []):
+                    if item['lookup'] != 'in':
+                        filters[f'{item["field"]}__{item["lookup"]}'] = item['value']
+                    else:
+                        filters[f'{item["field"]}__{item["lookup"]}'] = [value.strip() for value in item['value'].split(',')]
 
         return filter_users(filters)
 

admin/templates/notifications/notification_campaigns_detail.html

@@ -40,7 +40,7 @@ <h4>Progress</h4>
         style="display:inline;"
     >
         {% csrf_token %}
-        <button type="submit" class="btn btn-primary" {% if notification_campaign.status == "completed" %}disabled{% endif %}>
+        <button type="submit" class="btn btn-primary" {% if notification_campaign.status != "created" %}disabled{% endif %}>
             Start Campaign
         </button>
     </form>
@@ -74,14 +74,15 @@ <h4>General</h4>
                             </a>
                         </td>
                         <td>
+
                             <form
                                 id="restart-campaign-form"
                                 method="post"
                                 action="{% url 'notifications:notification_campaigns_start' notification_campaign.id %}?restart_failed=true"
                                 style="display:inline;"
                             >
                                 {% csrf_token %}
-                                <button type="submit" class="btn btn-primary">
+                                <button type="submit" class="btn btn-primary" {% if notification_campaign.status != "partially_completed" %}disabled{% endif %}>
                                     Restart Campaign
                                 </button>
                             </form>

osf/email/notification_campaign.py

@@ -146,11 +146,13 @@ def start_notification_campaign(campaign_id, restart_failed=False):
     if predefined_filter_name := filters.get('predefined'):
         filters = FILTER_PRESETS.get(predefined_filter_name, {})
     else:
-        filters = {
-            f'{item["field"]}__{item["lookup"]}': item['value']
-            for item in filters.get('manual', [])
-        }
-
+        manual_filters = {}
+        for item in filters.get('manual', []):
+            if item['lookup'] != 'in':
+                manual_filters[f'{item["field"]}__{item["lookup"]}'] = item['value']
+            else:
+                manual_filters[f'{item["field"]}__{item["lookup"]}'] = [value.strip() for value in item['value'].split(',')]
+        filters = manual_filters
     tasks = []
     total_recipients = 0
     batch_size = campaign.metadata.get('execution', {}).get('batch_size', 1000)