- emotion을 사용한만큼 color.js를 바로 불러오기보다는 ThemeProvider를 만들어서 사용하는것을 추천
- 회원가입시 코드가 되게 중복된형태가 많아지는것 같습니다. 코드를 좀 더 줄일 수 있도록 하면 좋을 것 같아요.
```jsx
const [values, setValues] = useState({ email: “”, password: “” });
const [errors, setErrors] = useState({});
```
- styles과 component의 분리가 필요해 보입니다.
- 페이지에서만 필요한 상수등이 있다면 그 부분 또한 별도 파일로 분리하여 관리해주세요.
```jsx
/SignUp/index.js
/SignUp/styles.js
```
- MyGroupd 페이지의 경우 페이지내 컨텍스트API가 적용되어있던데, 꼭 필요한 부분인건가요??
- isLoading 여부에 따른 리턴은 분기처리보다는 아래와 같은 형태를 개인적으로 추천드립니다.
```jsx
if(isLoading){
return(<Loading/>)
}return(
<div>
.
.
.
</div>
)
```
- last를 판단하는 css구문이 보이던데 `li:not(:last-child) {` 와 같이 사용할수는 없는걸까요?
- `domain`폴더에 `groupInfo` 소문자로 시작하는 폴더가 존재합니다.
- axios에서 에러가 발생한다면 발생한 에러메시지를 받아서 처리하여 화면등에 표춣시키는 구문또한 필요할것 같습니다. 현재는 console로만 처리되어 있네요 :)
- `...props`에 대해서
- 기본적으로 제공하는 태그를 기반으로 만드는 core한 컴포넌트라면 필요할 수 있지만, 그 이외에 대해서는 사용중인 props를 모두 정의하여 사용하는것이 좋을것 같습니다.