exclude email from update user if not in data

GCTC-NTGC · Mar 29, 2023 · ad02f32 · ad02f32
1 parent ae28900
commit ad02f32
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/apps/web/src/pages/Users/UpdateUserPage/UpdateUserPage.tsx b/apps/web/src/pages/Users/UpdateUserPage/UpdateUserPage.tsx
@@ -311,7 +311,9 @@ const UpdateUserPage = () => {
       id,
       user: {
         id,
-        email: emptyToNull(data.email),
+        // Do not include email in the request if it is not part of form data
+        // to prevent accidentally setting it to null
+        email: data.email !== undefined ? emptyToNull(data.email) : undefined,
         ...pick(data, [
           "firstName",
           "lastName",