feat!: [#591] rename enum variant EmailOnSignup::None to Ignored

Author: josecelano

src/config/v1/auth.rs

@@ -58,7 +58,7 @@ pub enum EmailOnSignup {
     /// The email is optional on signup.
     Optional,
     /// The email is not allowed on signup. It will only be ignored if provided.
-    None, // code-review: rename to `Ignored`?
+    Ignored,
 }
 
 impl Default for EmailOnSignup {
@@ -72,7 +72,7 @@ impl fmt::Display for EmailOnSignup {
         let display_str = match self {
             EmailOnSignup::Required => "required",
             EmailOnSignup::Optional => "optional",
-            EmailOnSignup::None => "none",
+            EmailOnSignup::Ignored => "ignored",
         };
         write!(f, "{display_str}")
     }
@@ -85,8 +85,10 @@ impl FromStr for EmailOnSignup {
         match s.to_lowercase().as_str() {
             "required" => Ok(EmailOnSignup::Required),
             "optional" => Ok(EmailOnSignup::Optional),
-            "none" => Ok(EmailOnSignup::None),
-            _ => Err(format!("Unknown config 'email_on_signup' option (required, optional, none): {s}")),
+            "none" => Ok(EmailOnSignup::Ignored),
+            _ => Err(format!(
+                "Unknown config 'email_on_signup' option (required, optional, ignored): {s}"
+            )),
         }
     }
 }

src/services/user.rs

@@ -86,7 +86,7 @@ impl RegistrationService {
                 }
                 registration_form.email.clone()
             }
-            EmailOnSignup::None => None,
+            EmailOnSignup::Ignored => None,
             EmailOnSignup::Optional => registration_form.email.clone(),
         };