From 4b90c4647667de404b5a36bd17549ebee0109b5d Mon Sep 17 00:00:00 2001
From: Tim Pechersky <tim.pechersky@gmail.com>
Date: Thu, 23 Sep 2021 14:57:29 +0200
Subject: [PATCH 1/6] Access tokens - API tokens

---
 frontend/src/components/AccountIconButton.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/components/AccountIconButton.js b/frontend/src/components/AccountIconButton.js
index 4dc74bb5..75ae94b1 100644
--- a/frontend/src/components/AccountIconButton.js
+++ b/frontend/src/components/AccountIconButton.js
@@ -36,7 +36,7 @@ const AccountIconButton = (props) => {
             <MenuItem>Security</MenuItem>
           </RouterLink>
           <RouterLink href="/account/tokens" passHref>
-            <MenuItem>Access tokens</MenuItem>
+            <MenuItem>API tokens</MenuItem>
           </RouterLink>
         </MenuGroup>
         <MenuDivider />

From 644699f86cb7f577552e6da1d9f31a9c5331149f Mon Sep 17 00:00:00 2001
From: Tim Pechersky <tim.pechersky@gmail.com>
Date: Thu, 23 Sep 2021 14:57:46 +0200
Subject: [PATCH 2/6] fix backend bug wrong token note

---
 backend/moonstream/routes/users.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/backend/moonstream/routes/users.py b/backend/moonstream/routes/users.py
index 97629ee8..fefd8058 100644
--- a/backend/moonstream/routes/users.py
+++ b/backend/moonstream/routes/users.py
@@ -188,7 +188,7 @@ async def token_update_handler(
 ) -> BugoutToken:
     token = request.state.token
     try:
-        response = bc.update_token(token, token_note=token_note)
+        response = bc.update_token(access_token, token_note=token_note)
     except BugoutResponseException as e:
         raise MoonstreamHTTPException(status_code=e.status_code, detail=e.detail)
     except Exception as e:

From 30241242a0a0c3f1a761fe1ff2b37ba6a45c185a Mon Sep 17 00:00:00 2001
From: Tim Pechersky <tim.pechersky@gmail.com>
Date: Thu, 23 Sep 2021 14:58:48 +0200
Subject: [PATCH 3/6] layout and ux improvements

---
 frontend/pages/account/tokens.js      | 10 +++----
 frontend/src/components/Sidebar.js    | 20 +++++++++++---
 frontend/src/components/TokensList.js | 39 ++++++++++++++++++---------
 frontend/src/core/hooks/useTokens.js  |  4 +--
 4 files changed, 50 insertions(+), 23 deletions(-)

diff --git a/frontend/pages/account/tokens.js b/frontend/pages/account/tokens.js
index 8b7db744..b5ba152f 100644
--- a/frontend/pages/account/tokens.js
+++ b/frontend/pages/account/tokens.js
@@ -24,7 +24,7 @@ const Tokens = () => {
   const { onOpen, onClose, isOpen } = useDisclosure();
   const [newToken, setNewToken] = useState(null);
   const [tokens, setTokens] = useState();
-  const { list, update, revoke, isLoading, data } = useTokens();
+  const { list, updateMutation, revoke, isLoading, data } = useTokens();
 
   useEffect(() => {
     list();
@@ -44,7 +44,7 @@ const Tokens = () => {
     if (data?.data?.user_id) {
       setTokens(data.data);
     }
-  }, [data?.data, isLoading]);
+  }, [data, isLoading]);
 
   useEffect(() => {
     document.title = `Tokens`;
@@ -75,8 +75,8 @@ const Tokens = () => {
               </ModalBody>
             </ModalContent>
           </Modal>
-          <Heading variant="tokensScreen"> My access tokens </Heading>
-          <VStack overflow="initial" maxH="unset" height="100%">
+          <Heading variant="tokensScreen"> My API tokens </Heading>
+          <VStack overflow="initial" maxH="unset" height="100%" maxW="100%">
             <Button
               alignSelf="flex-end"
               onClick={onOpen}
@@ -90,7 +90,7 @@ const Tokens = () => {
               data={tokens}
               revoke={revoke}
               isLoading={isLoading}
-              updateCallback={update}
+              update={updateMutation}
             />
           </VStack>
         </ScaleFade>
diff --git a/frontend/src/components/Sidebar.js b/frontend/src/components/Sidebar.js
index 238af760..5cc6f818 100644
--- a/frontend/src/components/Sidebar.js
+++ b/frontend/src/components/Sidebar.js
@@ -8,10 +8,15 @@ import {
 } from "react-pro-sidebar";
 import { useContext } from "react";
 import RouterLink from "next/link";
-import { Flex, Image, IconButton } from "@chakra-ui/react";
+import { Flex, Image, IconButton, Divider } from "@chakra-ui/react";
 import UIContext from "../core/providers/UIProvider/context";
 import React from "react";
-import { HamburgerIcon, ArrowLeftIcon, ArrowRightIcon } from "@chakra-ui/icons";
+import {
+  HamburgerIcon,
+  ArrowLeftIcon,
+  ArrowRightIcon,
+  LockIcon,
+} from "@chakra-ui/icons";
 import { MdTimeline, MdSettings } from "react-icons/md";
 import { ImStatsBars } from "react-icons/im";
 import { HiAcademicCap } from "react-icons/hi";
@@ -117,7 +122,16 @@ const Sidebar = () => {
         </SidebarContent>
       )}
 
-      <SidebarFooter></SidebarFooter>
+      <SidebarFooter style={{ paddingBottom: "3rem" }}>
+        <Divider color="gray.300" w="100%" />
+        {ui.isLoggedIn && (
+          <Menu iconShape="square">
+            <MenuItem icon={<LockIcon />}>
+              <RouterLink href="/account/tokens">API Tokens</RouterLink>
+            </MenuItem>
+          </Menu>
+        )}
+      </SidebarFooter>
     </ProSidebar>
   );
 };
diff --git a/frontend/src/components/TokensList.js b/frontend/src/components/TokensList.js
index 18ba1ec1..81297249 100644
--- a/frontend/src/components/TokensList.js
+++ b/frontend/src/components/TokensList.js
@@ -15,9 +15,12 @@ import { DeleteIcon } from "@chakra-ui/icons";
 import moment from "moment";
 import CopyButton from "./CopyButton";
 
-const List = ({ data, revoke, isLoading, updateCallback }) => {
+const List = ({ data, revoke, isLoading, update }) => {
   const userToken = localStorage.getItem("BUGOUT_ACCESS_TOKEN");
 
+  const cellProps = {
+    px: ["2px", "6px", "inherit"],
+  };
   if (data) {
     return (
       <Table
@@ -30,10 +33,10 @@ const List = ({ data, revoke, isLoading, updateCallback }) => {
       >
         <Thead>
           <Tr>
-            <Th>Token</Th>
-            <Th>Date Created</Th>
-            <Th>Note</Th>
-            <Th>Actions</Th>
+            <Th>Label</Th>
+            <Th {...cellProps}>Token</Th>
+            <Th {...cellProps}>Date Created</Th>
+            <Th {...cellProps}>Actions</Th>
           </Tr>
         </Thead>
         <Tbody>
@@ -42,27 +45,37 @@ const List = ({ data, revoke, isLoading, updateCallback }) => {
               if (userToken !== token.id) {
                 return (
                   <Tr key={`token-row-${token.id}`}>
-                    <Td mr={4} p={0}>
-                      <CopyButton>{token.id}</CopyButton>
-                    </Td>
-                    <Td py={0}>{moment(token.created_at).format("L")}</Td>
-                    <Td py={0}>
+                    <Td py={0} {...cellProps}>
                       <Editable
                         colorScheme="blue"
-                        placeholder="enter note here"
+                        placeholder="Click to set up label"
                         defaultValue={token.note}
+                        isDisabled={update.isLoading}
                         onSubmit={(nextValue) =>
-                          updateCallback({ token: token.id, note: nextValue })
+                          update.mutate({ token: token.id, note: nextValue })
                         }
                       >
                         <EditablePreview
                           maxW="40rem"
+                          textColor={token.note ? "inherit" : "gray.900"}
                           _placeholder={{ color: "black" }}
                         />
                         <EditableInput maxW="40rem" />
                       </Editable>
                     </Td>
-                    <Td py={0}>
+                    <Td
+                      mr={4}
+                      py={0}
+                      {...cellProps}
+                      isTruncated
+                      maxW={["100px", "150px", "300px"]}
+                    >
+                      <CopyButton>{token.id}</CopyButton>
+                    </Td>
+                    <Td py={0} {...cellProps}>
+                      {moment(token.created_at).format("L")}
+                    </Td>
+                    <Td py={0} {...cellProps}>
                       <IconButton
                         size="sm"
                         variant="ghost"
diff --git a/frontend/src/core/hooks/useTokens.js b/frontend/src/core/hooks/useTokens.js
index 733ac58a..14322507 100644
--- a/frontend/src/core/hooks/useTokens.js
+++ b/frontend/src/core/hooks/useTokens.js
@@ -20,7 +20,7 @@ const useTokens = () => {
     },
   });
 
-  const { mutate: update } = useMutation(AuthService.updateToken, {
+  const updateMutation = useMutation(AuthService.updateToken, {
     onSuccess: () => {
       list();
     },
@@ -42,7 +42,7 @@ const useTokens = () => {
   return {
     createToken,
     list,
-    update,
+    updateMutation,
     revoke,
     isLoading,
     data,

From 1283896a30361dc2b603dc6e43d5808443d22ec6 Mon Sep 17 00:00:00 2001
From: Tim Pechersky <tim.pechersky@gmail.com>
Date: Thu, 23 Sep 2021 15:02:21 +0200
Subject: [PATCH 4/6] filter for current frontend token

---
 frontend/src/components/TokensList.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/components/TokensList.js b/frontend/src/components/TokensList.js
index 81297249..6e4bdbc8 100644
--- a/frontend/src/components/TokensList.js
+++ b/frontend/src/components/TokensList.js
@@ -16,7 +16,7 @@ import moment from "moment";
 import CopyButton from "./CopyButton";
 
 const List = ({ data, revoke, isLoading, update }) => {
-  const userToken = localStorage.getItem("BUGOUT_ACCESS_TOKEN");
+  const userToken = localStorage.getItem("MOONSTREAM_ACCESS_TOKEN");
 
   const cellProps = {
     px: ["2px", "6px", "inherit"],

From c3d1f50806a7c880cb29acf4ab3d35da996cb03f Mon Sep 17 00:00:00 2001
From: Tim Pechersky <tim.pechersky@gmail.com>
Date: Thu, 23 Sep 2021 15:07:14 +0200
Subject: [PATCH 5/6] corrected argument

---
 backend/moonstream/routes/users.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/backend/moonstream/routes/users.py b/backend/moonstream/routes/users.py
index fefd8058..6a04501b 100644
--- a/backend/moonstream/routes/users.py
+++ b/backend/moonstream/routes/users.py
@@ -188,7 +188,7 @@ async def token_update_handler(
 ) -> BugoutToken:
     token = request.state.token
     try:
-        response = bc.update_token(access_token, token_note=token_note)
+        response = bc.update_token(token=access_token, token_note=token_note)
     except BugoutResponseException as e:
         raise MoonstreamHTTPException(status_code=e.status_code, detail=e.detail)
     except Exception as e:

From 0927a24ae77fb9cb08905794fad55a8a4243ee94 Mon Sep 17 00:00:00 2001
From: Tim Pechersky <tim.pechersky@gmail.com>
Date: Thu, 23 Sep 2021 15:08:07 +0200
Subject: [PATCH 6/6] remove unused definition

---
 backend/moonstream/routes/users.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/backend/moonstream/routes/users.py b/backend/moonstream/routes/users.py
index 6a04501b..c414fabf 100644
--- a/backend/moonstream/routes/users.py
+++ b/backend/moonstream/routes/users.py
@@ -186,7 +186,6 @@ async def tokens_handler(request: Request) -> BugoutUserTokens:
 async def token_update_handler(
     request: Request, token_note: str = Form(...), access_token: str = Form(...)
 ) -> BugoutToken:
-    token = request.state.token
     try:
         response = bc.update_token(token=access_token, token_note=token_note)
     except BugoutResponseException as e: