BugFix on Profile Page

[fiskryeziu]

Description

This pull request fixes errors (#725) cannot read properties of null of displayName, uid. The error occurs when the user object or auth is still null or undefined, and the code tries to access the displayName & uid property.

##Changes Made
1.Added optional chaining before accessing displayName.
2.Added conditional check in useEffect to make sure auth.currentUser it's not null.
3.Removed unnecessary useEffect

  useEffect(() => {
    if (auth.currentUser) {
      setUser(auth.currentUser);
    } else {
      navigate("/dummygram/login");
    }
  }, []);

because was doing the same thing like below useEffect (which this handles much better the auth state changes)

useEffect(() => {
    const unsubscribe = auth.onAuthStateChanged((authUser) => {
      if (authUser) {
        setUser(authUser);
        setName(location?.state?.name || authUser.displayName);
        setAvatar(location?.state?.avatar || authUser.photoURL);
        setEmail(
          location?.state?.name === authUser?.displayName
            ? location?.state?.email || authUser.email
            : ""
        );
        setUid(location?.state?.uid || authUser.uid);
      } else {
        navigate("/dummygram/login");
      }
    });

    return () => {
      unsubscribe();
    };
  }, []);

4.removed profilePc & adde avatar state inside src prop to display user profile img.

[narayan954]

looks good to me!

[narayan954]

Description

This pull request fixes errors (#725) cannot read properties of null of displayName, uid. The error occurs when the user object or auth is still null or undefined, and the code tries to access the displayName & uid property.

##Changes Made 1.Added optional chaining before accessing displayName. 2.Added conditional check in useEffect to make sure auth.currentUser it's not null. 3.Removed unnecessary useEffect

  useEffect(() => {
    if (auth.currentUser) {
      setUser(auth.currentUser);
    } else {
      navigate("/dummygram/login");
    }
  }, []);

because was doing the same thing like below useEffect (which this handles much better the auth state changes)

useEffect(() => {
    const unsubscribe = auth.onAuthStateChanged((authUser) => {
      if (authUser) {
        setUser(authUser);
        setName(location?.state?.name || authUser.displayName);
        setAvatar(location?.state?.avatar || authUser.photoURL);
        setEmail(
          location?.state?.name === authUser?.displayName
            ? location?.state?.email || authUser.email
            : ""
        );
        setUid(location?.state?.uid || authUser.uid);
      } else {
        navigate("/dummygram/login");
      }
    });

    return () => {
      unsubscribe();
    };
  }, []);

4.removed profilePc & adde avatar state inside src prop to display user profile img.

thanks a lot for fixing these many stuffs @fiskryeziu , merging them!