chore: address PR comments

src/components/common/PinButton.js

@@ -16,39 +16,26 @@ const PinButton = ({ item }) => {
   const [isPinned, setPinned] = useState(item?.settings?.isPinned);
 
   const handlePin = () => {
-    const { settings } = item;
-    settings.isPinned = true;
+    const newState = !isPinned;
 
-    editItem.mutate({
-      id: item.id,
-      // use item login tag id
-      name: item.name,
-      settings: item.settings,
-    });
-
-    setPinned(true);
-  };
+    setPinned(newState);
 
-  const handleUnpin = () => {
     const { settings } = item;
-    settings.isPinned = false;
+    settings.isPinned = newState;
 
     editItem.mutate({
       id: item.id,
-      // use item login tag id
       name: item.name,
       settings: item.settings,
     });
-
-    setPinned(false);
   };
 
   return (
     <Tooltip title={isPinned ? t('Unpin') : t('Pin')}>
       <IconButton
-        aria-label="favorite"
+        aria-label={isPinned ? t('Unpin') : t('Pin')}
         className={PIN_ITEM_BUTTON_CLASS}
-        onClick={isPinned ? handleUnpin : handlePin}
+        onClick={handlePin}
       >
         {isPinned ? (
           <PushPinIcon fontSize="small" />

src/components/item/settings/ItemSettings.js

@@ -65,7 +65,7 @@ const ItemSettings = ({ item }) => {
 
       <FormGroup>
         <FormControlLabel
-          label="Pin Item"
+          label={t('Pin')}
           control={(
             <Switch
               id={SETTINGS_PINNED_TOGGLE}
@@ -76,7 +76,7 @@ const ItemSettings = ({ item }) => {
           )}
         />
         <FormControlLabel
-          label="Show Chatbox"
+          label={t('Show Chat')}
           control={(
             <Switch
               id={SETTINGS_CHATBOX_TOGGLE}

src/components/main/Home.js

@@ -2,7 +2,6 @@ import React from 'react';
 import PropTypes from 'prop-types';
 import { useTranslation } from 'react-i18next';
 import { withRouter } from 'react-router';
-import { List } from 'immutable';
 import { hooks } from '../../config/queryClient';
 import ItemHeader from '../item/header/ItemHeader';
 import Items from './Items';
@@ -33,7 +32,7 @@ const Home = () => {
       <Items
         id={OWNED_ITEMS_ID}
         title={t('My Items')}
-        items={List(ownItems)}
+        items={ownItems}
         headerElements={[<NewItemButton fontSize="small" />]}
       />
     </Main>

src/langs/en.json

@@ -128,6 +128,9 @@
     "Authenticated Members": "Authenticated Members",
     "Username and Password": "Username and Password",
     "Username": "Username",
-    "No user has authenticated to this item yet.": "No user has authenticated to this item yet."
+    "No user has authenticated to this item yet.": "No user has authenticated to this item yet.",
+    "Pin": "Pin this item",
+    "Unpin": "Unpin this item",
+    "Show Chat": "Show Chat"
   }
 }

src/langs/fr.json

@@ -128,6 +128,9 @@
     "Authenticated Members": "Membres Connectés",
     "Username and Password": "Nom d'utilisateur et Mot de passe",
     "Username": "Nom d'utilisateur",
-    "No user has authenticated to this item yet.": "Aucun utilisateur ne s'est encore connecté à cet élément."
+    "No user has authenticated to this item yet.": "Aucun utilisateur ne s'est encore connecté à cet élément.",
+    "Pin": "Epingler cet élément",
+    "Unpin": "Détacher cet élément",
+    "Show Chat": "Afficher le chat"
   }
 }