Skip to content

Commit

Permalink
Removed unnecessary prop expansion
Browse files Browse the repository at this point in the history
  • Loading branch information
wrigh393 committed Mar 17, 2021
1 parent 9cf0301 commit 39ed274
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/components/UserAvatar/UserAvatar.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@ import React, { useEffect, useState } from "react";
import PropTypes from "prop-types";
import { Avatar } from "@material-ui/core";

export default function UserAvatar({ name, alias, ...avatarProps }) {
export default function UserAvatar({ name, alias }) {
const [userImage, setUserImage] = useState([]);

useEffect(() => {
const getImage = async () => {
const response = await fetch(`https://engineering.purdue.edu/ECN/PersonPhotos/getPhoto?json=1&alias=${alias}`);
Expand All @@ -15,7 +15,7 @@ export default function UserAvatar({ name, alias, ...avatarProps }) {
}, [alias])

return (
<Avatar {...avatarProps} src={`https://engineering.purdue.edu/${userImage.imagePath}`}>
<Avatar src={`https://engineering.purdue.edu/${userImage.imagePath}`}>
{name === "" ? null : name.charAt(0)}
</Avatar>
);
Expand Down

0 comments on commit 39ed274

Please sign in to comment.