fix(users): set preferred notification frequency on signup (#1630)

* fix(local-auth-helpers): set user notification frequency from request body

Author: henryk1229

src/containers/UserEdit/index.jsx

@@ -4,9 +4,6 @@ import Dialog from "@material-ui/core/Dialog";
 import DialogActions from "@material-ui/core/DialogActions";
 import DialogContent from "@material-ui/core/DialogContent";
 import DialogTitle from "@material-ui/core/DialogTitle";
-import InputLabel from "@material-ui/core/InputLabel";
-import MenuItem from "@material-ui/core/MenuItem";
-import Select from "@material-ui/core/Select";
 import { css, StyleSheet } from "aphrodite";
 import PropTypes from "prop-types";
 import queryString from "query-string";
@@ -170,15 +167,6 @@ class UserEdit extends React.Component {
     this.setState({ changePasswordDialog: false, successDialog: false });
   };
 
-  handleNotificationFrequencyChange = (event) => {
-    const newFrequency = event.target.value;
-    const user = {
-      ...this.state.user,
-      notificationFrequency: newFrequency
-    };
-    this.setState({ user });
-  };
-
   openSuccessDialog = () => this.setState({ successDialog: true });
 
   buildFormSchema = (authType) => {
@@ -297,27 +285,18 @@ class UserEdit extends React.Component {
                 name="cell"
                 {...dataTest("cell")}
               />
-              <div style={{ marginTop: 20 }}>
-                <InputLabel
-                  style={{ marginBottom: 15 }}
-                  id="notification-frequency-label"
-                >
-                  Notification Frequency
-                </InputLabel>
-                <Select
-                  id="notification-frequency"
-                  labelId="notification-frequency-label"
-                  name="notificationFrequency"
-                  value={user.notificationFrequency}
-                  onChange={this.handleNotificationFrequencyChange}
-                >
-                  {Object.values(NotificationFrequencyType).map((option) => (
-                    <MenuItem key={option} value={option}>
-                      {titleCase(option)}
-                    </MenuItem>
-                  ))}
-                </Select>
-              </div>
+              <SpokeFormField
+                label="Notification Frequency"
+                name="notificationFrequency"
+                {...dataTest("notificationFrequency")}
+                type="select"
+                choices={Object.values(NotificationFrequencyType).map(
+                  (option) => ({
+                    value: option,
+                    label: titleCase(option)
+                  })
+                )}
+              />
             </span>
           )}
           {(authType === UserEditMode.Login ||

src/server/local-auth-helpers.ts

@@ -161,7 +161,8 @@ const signup: AuthHelper = async (options) => {
           first_name: capitalizeWord(reqBody.firstName),
           last_name: capitalizeWord(reqBody.lastName),
           cell: reqBody.cell,
-          is_superadmin: false
+          is_superadmin: false,
+          notification_frequency: reqBody.notificationFrequency
         })
         .returning("*");
       resolve(user);