DaleStudy / leaderboard

달레 스터디 리더보드
http://leaderboard.dalestudy.com/
3 stars 0 forks source link

feat : add Table component #58

Closed SamTheKorean closed 6 days ago

SamTheKorean commented 2 weeks ago

체크리스트

스크린샷

Screenshot 2024-11-08 at 12 04 40 AM
SamTheKorean commented 2 weeks ago

고칠 점이 많지만 진행상황 공유 차원에서 먼저 draft pr올렸습니다! 코드 전반적으로 직접 검토하여 amend할 예정이지만 typescript와 리엑트 코드가 처음이라 방향성 자체에 피드백이 있으시다면 공유 부탁드립니다!

sounmind commented 2 weeks ago

고칠 점이 많지만 진행상황 공유 차원에서 먼저 draft pr올렸습니다! 코드 전반적으로 직접 검토하여 amend할 예정이지만 typescript와 리엑트 코드가 처음이라 방향성 자체에 피드백이 있으시다면 공유 부탁드립니다!

좋은 태도네요! test.todo 메서드를 사용해서 이 컴포넌트에서 어떤 부분이 테스트 되어야 하는지 미리 메모해두는 것도 리뷰하는 사람이 코드 작성자가 어떤 의도로 컴포넌트를 구현하려는 것인지 아는데 도움이 될 것 같아요ㅎㅎ

SamTheKorean commented 1 week ago

승인 감사합니다! 병합은 data fetching hook으로 업데이트 한 뒤 진행하겠습니다!

SamTheKorean commented 6 days ago

우선 모의 데이터로 병합하고 페이지 완성 pr에서 실제 데이터 연결하겠습니다!