From 74d2cd8d03b3d40e5f9548a5680818cb79f6f3b1 Mon Sep 17 00:00:00 2001
From: Brian Cao <brian@umami.is>
Date: Wed, 28 Dec 2022 12:37:09 -0800
Subject: [PATCH] Feat/um 140 password modal (#1718)

* Change password to modal.

* Close on save.
---
 components/forms/UserPasswordForm.js          |  5 +--
 components/pages/Dashboard.js                 |  4 +--
 components/pages/ProfileSettings.js           | 27 ++++++++++++++--
 components/pages/UserSettings.js              | 31 +++++++++++++++++--
 .../{WebsiteList.js => WebsiteChartList.js}   |  0
 components/settings/ProfileDetails.js         |  2 +-
 ...s.module.css => ProfileDetails.module.css} |  0
 7 files changed, 58 insertions(+), 11 deletions(-)
 rename components/pages/{WebsiteList.js => WebsiteChartList.js} (100%)
 rename components/settings/{ProfileSettings.module.css => ProfileDetails.module.css} (100%)

diff --git a/components/forms/UserPasswordForm.js b/components/forms/UserPasswordForm.js
index 748f462b..c28d9624 100644
--- a/components/forms/UserPasswordForm.js
+++ b/components/forms/UserPasswordForm.js
@@ -6,7 +6,7 @@ import { getClientAuthToken } from 'lib/client';
 import styles from './UserPasswordForm.module.css';
 import useUser from 'hooks/useUser';
 
-export default function UserPasswordForm({ onSave, userId }) {
+export default function UserPasswordForm({ onSave, onClose, userId }) {
   const {
     user: { id },
   } = useUser();
@@ -68,9 +68,10 @@ export default function UserPasswordForm({ onSave, userId }) {
         <PasswordField autoComplete="off" />
       </FormInput>
       <FormButtons flex>
-        <Button type="submit" disabled={isLoading}>
+        <Button type="submit" variant="primary" disabled={isLoading}>
           Save
         </Button>
+        <Button onClick={onClose}>Close</Button>
       </FormButtons>
     </Form>
   );
diff --git a/components/pages/Dashboard.js b/components/pages/Dashboard.js
index 343ac3f7..452288dd 100644
--- a/components/pages/Dashboard.js
+++ b/components/pages/Dashboard.js
@@ -2,7 +2,7 @@ import { useState } from 'react';
 import { defineMessages, useIntl } from 'react-intl';
 import Page from 'components/layout/Page';
 import PageHeader from 'components/layout/PageHeader';
-import WebsiteList from 'components/pages/WebsiteList';
+import WebsiteChartList from 'components/pages/WebsiteChartList';
 import { Button } from 'react-basics';
 import DashboardSettingsButton from 'components/settings/DashboardSettingsButton';
 import useFetch from 'hooks/useFetch';
@@ -37,7 +37,7 @@ export default function Dashboard({ userId }) {
         {!editing && <DashboardSettingsButton />}
       </PageHeader>
       {editing && <DashboardEdit websites={data} />}
-      {!editing && <WebsiteList websites={data} showCharts={showCharts} limit={max} />}
+      {!editing && <WebsiteChartList websites={data} showCharts={showCharts} limit={max} />}
       {max < data.length && (
         <Button className={styles.button} onClick={handleMore}>
           {formatMessage(messages.more)}
diff --git a/components/pages/ProfileSettings.js b/components/pages/ProfileSettings.js
index ab6dbb8a..cb9dd027 100644
--- a/components/pages/ProfileSettings.js
+++ b/components/pages/ProfileSettings.js
@@ -2,15 +2,27 @@ import Page from 'components/layout/Page';
 import PageHeader from 'components/layout/PageHeader';
 import ProfileDetails from 'components/settings/ProfileDetails';
 import { useState } from 'react';
-import { Breadcrumbs, Item, Tabs, useToast } from 'react-basics';
+import { Breadcrumbs, Item, Tabs, useToast, Modal, Button } from 'react-basics';
 import UserPasswordForm from 'components/forms/UserPasswordForm';
+import Icon from './../common/Icon';
+import Pen from 'assets/pen.svg';
 
 export default function ProfileSettings() {
+  const [edit, setEdit] = useState(false);
   const [tab, setTab] = useState('general');
   const { toast, showToast } = useToast();
 
   const handleSave = () => {
     showToast({ message: 'Saved successfully.', variant: 'success' });
+    setEdit(false);
+  };
+
+  const handleAdd = () => {
+    setEdit(true);
+  };
+
+  const handleClose = () => {
+    setEdit(false);
   };
 
   return (
@@ -20,13 +32,22 @@ export default function ProfileSettings() {
         <Breadcrumbs>
           <Item>Profile</Item>
         </Breadcrumbs>
+        <Button onClick={handleAdd}>
+          <Icon>
+            <Pen />
+          </Icon>
+          Change Password
+        </Button>
       </PageHeader>
       <Tabs selectedKey={tab} onSelect={setTab} style={{ marginBottom: 30, fontSize: 14 }}>
         <Item key="general">General</Item>
-        <Item key="password">Password</Item>
       </Tabs>
       {tab === 'general' && <ProfileDetails />}
-      {tab === 'password' && <UserPasswordForm onSave={handleSave} />}
+      {edit && (
+        <Modal title="Add website" onClose={handleClose}>
+          {close => <UserPasswordForm onSave={handleSave} onClose={close} />}
+        </Modal>
+      )}
     </Page>
   );
 }
diff --git a/components/pages/UserSettings.js b/components/pages/UserSettings.js
index b74164a9..f53678ce 100644
--- a/components/pages/UserSettings.js
+++ b/components/pages/UserSettings.js
@@ -9,9 +9,11 @@ import useApi from 'hooks/useApi';
 import Link from 'next/link';
 import { useRouter } from 'next/router';
 import { useEffect, useState } from 'react';
-import { Breadcrumbs, Item, Tabs, useToast } from 'react-basics';
+import { Breadcrumbs, Item, Icon, Tabs, useToast, Modal, Button } from 'react-basics';
+import Pen from 'assets/pen.svg';
 
 export default function UserSettings({ userId }) {
+  const [edit, setEdit] = useState(false);
   const [values, setValues] = useState(null);
   const [tab, setTab] = useState('general');
   const { get } = useApi(getClientAuthToken());
@@ -32,6 +34,18 @@ export default function UserSettings({ userId }) {
     if (data) {
       setValues(state => ({ ...state, ...data }));
     }
+
+    if (edit) {
+      setEdit(false);
+    }
+  };
+
+  const handleAdd = () => {
+    setEdit(true);
+  };
+
+  const handleClose = () => {
+    setEdit(false);
   };
 
   const handleDelete = async () => {
@@ -55,15 +69,26 @@ export default function UserSettings({ userId }) {
           </Item>
           <Item>{values?.username}</Item>
         </Breadcrumbs>
+        <Button onClick={handleAdd}>
+          <Icon>
+            <Pen />
+          </Icon>
+          Change Password
+        </Button>
       </PageHeader>
       <Tabs selectedKey={tab} onSelect={setTab} style={{ marginBottom: 30, fontSize: 14 }}>
         <Item key="general">General</Item>
-        <Item key="password">Password</Item>
         <Item key="delete">Danger Zone</Item>
       </Tabs>
       {tab === 'general' && <UserEditForm userId={userId} data={values} onSave={handleSave} />}
-      {tab === 'password' && <UserPasswordForm userId={userId} data={values} onSave={handleSave} />}
       {tab === 'delete' && <UserDelete userId={userId} onSave={handleDelete} />}
+      {edit && (
+        <Modal title="Add website" onClose={handleClose}>
+          {close => (
+            <UserPasswordForm userId={userId} data={values} onSave={handleSave} onClose={close} />
+          )}
+        </Modal>
+      )}
     </Page>
   );
 }
diff --git a/components/pages/WebsiteList.js b/components/pages/WebsiteChartList.js
similarity index 100%
rename from components/pages/WebsiteList.js
rename to components/pages/WebsiteChartList.js
diff --git a/components/settings/ProfileDetails.js b/components/settings/ProfileDetails.js
index c77b946b..a8d37595 100644
--- a/components/settings/ProfileDetails.js
+++ b/components/settings/ProfileDetails.js
@@ -4,7 +4,7 @@ import React from 'react';
 import { FormattedMessage } from 'react-intl';
 import DateRangeSetting from './DateRangeSetting';
 import LanguageSetting from './LanguageSetting';
-import styles from './ProfileSettings.module.css';
+import styles from './ProfileDetails.module.css';
 import ThemeSetting from './ThemeSetting';
 
 export default function ProfileDetails() {
diff --git a/components/settings/ProfileSettings.module.css b/components/settings/ProfileDetails.module.css
similarity index 100%
rename from components/settings/ProfileSettings.module.css
rename to components/settings/ProfileDetails.module.css