Add check for REQUIRES_LOGIN when ASSIGN_ROLE provided

1 files changed, 6 insertions, 2 deletions

diff --git a/backend/models/form.py b/backend/models/form.py
index 5aacc8b..d95cabd 100644
--- a/backend/models/form.py
+++ b/backend/models/form.py
@@ -47,8 +47,12 @@ class Form(BaseModel):
         if any(v not in allowed_values for v in value):
             raise ValueError("Form features list contains one or more invalid values.")
 
-        if FormFeatures.COLLECT_EMAIL in value and FormFeatures.REQUIRES_LOGIN not in value:  # noqa
-            raise ValueError("COLLECT_EMAIL feature require REQUIRES_LOGIN feature.")
+        if FormFeatures.REQUIRES_LOGIN not in value:
+            if FormFeatures.COLLECT_EMAIL in value:
+                raise ValueError("COLLECT_EMAIL feature require REQUIRES_LOGIN feature.")
+
+            if FormFeatures.ASSIGN_ROLE in value:
+                raise ValueError("ASSIGN_ROLE feature require REQUIRES_LOGIN feature.")
 
         return value