removed all warnings

Lodestone-Team · Apr 16, 2022 · 059c0f5 · 059c0f5
1 parent 972467a
commit 059c0f5
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 20 deletions.
diff --git a/src/components/Login.js b/src/components/Login.js
@@ -1,5 +1,5 @@
 import React, { useEffect, useState } from "react";
-import { Link, useNavigate } from "react-router-dom";
+import { Link } from "react-router-dom";
 import { auth, logInWithEmailAndPassword, signInWithGoogle } from "../firebase";
 import { useAuthState } from "react-firebase-hooks/auth";
 import "./Login.css";
@@ -8,8 +8,7 @@ import Card from "./Card";
 function Login() {
   const [email, setEmail] = useState("");
   const [password, setPassword] = useState("");
-  const [user, loading, error] = useAuthState(auth);
-  const navigate = useNavigate();
+  const [user, loading] = useAuthState(auth);
   useEffect(() => {
     if (loading) {
       // maybe trigger a loading screen

diff --git a/src/components/Register.js b/src/components/Register.js
@@ -12,7 +12,7 @@ function Register() {
   const [email, setEmail] = useState("");
   const [password, setPassword] = useState("");
   const [name, setName] = useState("");
-  const [user, loading, error] = useAuthState(auth);
+  const [user, loading] = useAuthState(auth);
   const register = () => {
     if (!name) alert("Please enter name");
     registerWithEmailAndPassword(name, email, password);

diff --git a/src/components/Reset.js b/src/components/Reset.js
@@ -1,14 +1,12 @@
 import React, { useEffect, useState } from "react";
 import { useAuthState } from "react-firebase-hooks/auth";
-import { useNavigate } from "react-router-dom";
 import { Link } from "react-router-dom";
 import Card from "./Card";
 import { auth, sendPasswordReset } from "../firebase";
 import "./Reset.css";
 function Reset() {
   const [email, setEmail] = useState("");
-  const [user, loading, error] = useAuthState(auth);
-  const navigate = useNavigate();
+  const [user, loading] = useAuthState(auth);
   useEffect(() => {
     if (loading) return;
     // if (user) navigate("/dashboard");

diff --git a/src/screens/Dashboard.js b/src/screens/Dashboard.js
@@ -31,21 +31,22 @@ export default function Dashboard() {
 
 
   /* Firebase authentication */
-  const [user, loading, error] = useAuthState(auth);
+  const [user, loading] = useAuthState(auth);
   const [name, setName] = useState("");
-  const fetchUserName = async () => {
-    try {
-      const q = query(collection(db, "users"), where("uid", "==", user?.uid));
-      const doc = await getDocs(q);
-      const data = doc.docs[0].data();
-      setName(data.name);
-    } catch (err) {
-      console.error(err);
-      alert("An error occured while fetching user data");
-    }
-  };
-
+
   useEffect(() => {
+    const fetchUserName = async () => {
+      try {
+        const q = query(collection(db, "users"), where("uid", "==", user?.uid));
+        const doc = await getDocs(q);
+        const data = doc.docs[0].data();
+        setName(data.name);
+      } catch (err) {
+        console.error(err);
+        alert("An error occured while fetching user data");
+      }
+    };
+
     if (loading) return;
     if (!user) {
       setName("Not logged in");