From 68d331fcb2a8158fecc5a1848410daf825b73e7e Mon Sep 17 00:00:00 2001
From: Frank Duncan <frankduncan@opentechstrategies.com>
Date: Wed, 3 Jul 2024 06:29:20 -0500
Subject: [PATCH] Remove None reviewers from validation for batch change

This is the same change as ee286ac5d, but for batch reviewer change.

Issue ots/clients/smex/hypha-tracker#2: Can't assign Staff
as reviewers of applications
---
 hypha/apply/funds/forms.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hypha/apply/funds/forms.py b/hypha/apply/funds/forms.py
index 43e77ce53..53df41501 100644
--- a/hypha/apply/funds/forms.py
+++ b/hypha/apply/funds/forms.py
@@ -426,7 +426,7 @@ class BatchUpdateReviewersForm(forms.Form):
         role_reviewers = [
             user
             for field, user in self.cleaned_data.items()
-            if field in self.role_fields
+            if field in self.role_fields and user
         ]
 
         # If any of the users match and are set to multiple roles, throw an error
-- 
GitLab