Merge pull request #6111 from GCTC-NTGC/bug_6103_prevent-unwanted-ema…

…il-null [Bug] Fix user role from removing email
GCTC-NTGC · Mar 30, 2023 · 74c65c9 · 74c65c9
2 parents 08902df + ad02f32
commit 74c65c9
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",