Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

코치님 리뷰 #107

Open
rebi13 opened this issue Aug 10, 2023 · 0 comments
Open

코치님 리뷰 #107

rebi13 opened this issue Aug 10, 2023 · 0 comments

Comments

@rebi13
Copy link
Member

rebi13 commented Aug 10, 2023

  1. 전체적인 UI 이슈는 크게 보이지 않으나,
    디자이너가 있었다면 좀 더 괜찮은 UI가 나왔을거 같음(색이 pop한거는 알겠는데 뭔가 어우러지는 느낌이 아님)
  2. 스타일드 컴포넌트를 컴포넌트내에 작성을 하셨는데, 이를
    컴포넌트명.styles.ts 로 빼서 관리하면 이후에 늘어나는 스타일드 컴포넌트를 좀 더 쉽게 관리가 가능
  3. import 구문을 정리하는것도 중요
  1. common 폴더내에 atom 단위에 컴포넌트가 있는걸로 확인이 됨. 이를 atoms 라는 폴더명으로 바꿔서, 각 폴더별로 좀더 구분을 하면 좋을거같음
  2. interfaces 라고 잘 분리를 하엿는데, 보통 @types 라는 폴더 만들어서 관리함. 정답은 없으나, 우테코 수료생들이 만든 달록 프로젝트 구조도 보면서 공부하면 좋을듯 -> https://github.com/woowacourse-teams/2022-dallog/tree/develop/frontend
  3. API 처리하는거 react-query 또는 swr로 마이그레이션 해보기
  4. Suspense 컴포넌트 사용하여 fallback 상태에서 로딩 컴포넌트 보여줘보기

그 이외에 다양한 기술을 잘 접목해서 사용한거 좋음 + tailwind 공부해서 사용한거 칭찬해

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants