석현, 호민
useQuery 분리 제안
수혁
- 페이지네이션 미구현
- 현재 전체 페이지가 개수를 파악할 수 없어서 일단 모든 게시글을 다불러온다.
- 그래서 로딩이 오래걸린다.
- 무한스크롤밖에 생각나지 않아서 페이지네이션을 어떻게 구현해야 할 지 모르겠다.
- db에 저장되어있는 게시글의 갯수를 파악하지 못해서 페이지번호를 어떻게 측정해야 할 지 모르겠다.
호민
- 메인 페이지 성능 변경
- Lint 랑 prettier를 좀 수정할까 합니다.
석현
- 반응형 도입해보면 좋을 것 같다.
- 메인페이지 참여자 목록을 화면이 작을 때 슬라이드처럼 접고 펼 수 있게 할 수 있도록
지호
- 기능은 X
- 레이아웃 변경작업 → Figma / 리뷰는 브랜치 옮겨서 dev돌려가지고 확인하면서 리뷰
- 테스크는 작게 나눠서 진행할 예정 → 이슈 올릴 때 어떤작업 진행할지 상세히 설명해놓고 진행
현진
- form 관련 정해진 내용있는가
- Error처리 할 때 입력값을 안넣었을 때 alert로 보여줬는데 이게 안좋아서 수정해볼 예정
컨벤션
- 개인적으로 간단한 주석은 처리해주는게 오히려 도움될 것 같다는 의견입니다. 리팩토링 과정에서 재작업 해야될 부분들이 계속 공유되면 좋겠는데 주석이 아예 없으면 오잉.. 하는 부분들이 있을 것 같아요.
- 배열의 이름은 가능하면 모두 -s , 로 잡아보는것도 좋을 것 같아요.
- CSS단위는 rem단위에 맞게 4, 8, 12, 16 이렇게 재정의해야할것같아요
- interface를 기본으로한다는 뭔가 어색한 느낌이에요 객체라면 interface, 그외라면 type으로 가는게 맞는것 같아요
- 상대경로와 절대경로를 적절하게 섞어써야 할 것 같아요. 같은 컴포넌트 내에서도 절대경로를 써버리니까 import문이 많이 더러워지네요,,
→ comman +
.
경로 뜨거든요- import rule을 ESlINT로 잡고 있는데 trivago/? prettier로 잡을 수 있다.
- react - thirdparty Library - 절대경로 - 상대경로