Fix warnings

Author: mslaursen

fastapi_forge/frontend/panels/model_editor_panel.py

@@ -448,13 +448,18 @@ def _on_select_field(self, selection: list[dict[str, Any]]) -> None:
             and name in ("password", "email")
         ):
             ui.notify(
-                f"Cannot edit {name} field in authentication model.", type="warning"
+                f"Cannot edit {name} field in authentication model.",
+                type="warning",
             )
             self._deselect_field()
             return
 
         if name == "id":
             self._deselect_field()
+            ui.notify(
+                "Cannot edit the 'id' field, it is automatically generated.",
+                type="warning",
+            )
             return
 
         state.selected_field = next(
@@ -488,22 +493,18 @@ def _handle_update_field(
         default_value: str | None = None,
         extra_kwargs: dict[str, Any] | None = None,
     ) -> None:
+        if not state.selected_model or not state.selected_field:
+            return
+
+        exclude_set = {"id"}
         if (
-            not state.selected_model
-            or not state.selected_field
-            or state.selected_field.name == "id"
+            state.selected_model
+            and state.selected_model.metadata.is_auth_model
+            and state.use_builtin_auth
         ):
-            return
+            exclude_set.update({"email", "password"})
 
-        if state.selected_model.metadata.is_auth_model and name == "password":
-            ui.notify(
-                "Cannot rename password field in authentication model.", type="negative"
-            )
-            return
-        if state.selected_model.metadata.is_auth_model and name == "email":
-            ui.notify(
-                "Cannot rename email field in authentication model.", type="negative"
-            )
+        if name in exclude_set:
             return
 
         if state.selected_field.name != name and self._field_name_exists(name):