react-hooks/exhaustive-deps

2 files changed, 5 insertions, 4 deletions

diff --git a/vnext/src/components/Avatar.js b/vnext/src/components/Avatar.js
index 64c16ead..dc2050ce 100644
--- a/vnext/src/components/Avatar.js
+++ b/vnext/src/components/Avatar.js
@@ -27,7 +27,7 @@ function Avatar(props) {
                 isMounted = false;
             };
         }
-    }, [props.user]);
+    }, [props.user, user.uid, user.uri]);
     return (
         <div className="Avatar" style={props.style}>
             <div className="msg-avatar">
diff --git a/vnext/src/components/UserInfo.js b/vnext/src/components/UserInfo.js
index 0012d2a8..19b2f82b 100644
--- a/vnext/src/components/UserInfo.js
+++ b/vnext/src/components/UserInfo.js
@@ -16,18 +16,19 @@ let isMounted;
 
 export default function UserInfo(props) {
     const [user, setUser] = useState({ uname: props.user, uid: 0 });
+    const { onUpdate } = props;
     useEffect(() => {
         isMounted = true;
         info(user.uname).then(response => {
             if (isMounted) {
                 setUser(response.data);
-                props.onUpdate && props.onUpdate(response.data);
+                onUpdate && onUpdate(response.data);
             }
         });
         return () => {
             isMounted = false;
         };
-    }, []);
+    }, [onUpdate, user.uname]);
     return (
         <>
             <div className="userinfo">
@@ -80,7 +81,7 @@ export function UserLink(props) {
                 isMounted = false;
             };
         }
-    }, [props.user]);
+    }, [props.user, user.uid, user.uri]);
     return (
         user.uid ?
             <Link key={user.uid} to={`/${user.uname}/`} className="info-avatar"><img src={user.avatar}/>{user.uname}</Link>