From 3bdf859c73acb07fa1da615775ee7ec2e9da1725 Mon Sep 17 00:00:00 2001
From: Berkeley Martinez <berkeleytrue@users.noreply.github.com>
Date: Thu, 18 Jan 2018 14:57:04 -0800
Subject: [PATCH] fix(ToggleBtn): Make name required

radio buttons must share a name property
---
 common/app/routes/Settings/Email-Setting.jsx               | 3 +++
 common/app/routes/Settings/Toggle-Button/Toggle-Button.jsx | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/common/app/routes/Settings/Email-Setting.jsx b/common/app/routes/Settings/Email-Setting.jsx
index 59e343d214..ced3cd62cc 100644
--- a/common/app/routes/Settings/Email-Setting.jsx
+++ b/common/app/routes/Settings/Email-Setting.jsx
@@ -80,6 +80,7 @@ export default function EmailSettings({
         </Col>
         <Col sm={ 4 }>
           <TB
+            name='monthly-email'
             onChange={ toggleMonthlyEmail }
             value={ sendMonthlyEmail }
           />
@@ -95,6 +96,7 @@ export default function EmailSettings({
         </Col>
         <Col sm={ 4 }>
           <TB
+            name='notifications-email'
             onChange={ toggleNotificationEmail }
             value={ sendNotificationEmail }
           />
@@ -110,6 +112,7 @@ export default function EmailSettings({
         </Col>
         <Col sm={ 4 }>
           <TB
+            name='quincy-email'
             onChange={ toggleQuincyEmail }
             value={ sendQuincyEmail }
           />
diff --git a/common/app/routes/Settings/Toggle-Button/Toggle-Button.jsx b/common/app/routes/Settings/Toggle-Button/Toggle-Button.jsx
index 0e68d58127..8e78c33ccc 100644
--- a/common/app/routes/Settings/Toggle-Button/Toggle-Button.jsx
+++ b/common/app/routes/Settings/Toggle-Button/Toggle-Button.jsx
@@ -5,7 +5,7 @@ import { ToggleButtonGroup as BSBG, ToggleButton as TB } from 'react-bootstrap';
 import ns from './ns.json';
 
 const propTypes = {
-  name: PropTypes.string,
+  name: PropTypes.string.isRequired,
   onChange: PropTypes.func.isRequired,
   value: PropTypes.bool.isRequired
 };