Closed Seo-Hyang closed 1 month ago
Name | Link |
---|---|
Latest commit | |
Latest deploy log | https://app.netlify.com/sites/coworkers-1/deploys/66fa2d05d35521caa2e5520b |
Deploy Preview | https://deploy-preview-161--coworkers-1.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
아직 컬러가 전체적으로 수정이안됐네요~!
이 부분 나중에 그룹장은 왕관 표시 뜨고 나머지 그룹원들은 그룹 나가기 아이콘(적당한 아이콘으로..) 뜨게 해주세요! 그리고 현재는 케밥을 누르면 프로필 모달이 뜨는데 저 컴포넌트를 눌렀을때 모달이 뜨게끔 수정해주시면 좋을 거 같습니다!
p4) 그리고 전체적으로 라이트 모드에서 구분이 안되는 부분이 있으면 shadow 활용해주세요~!
페이지네이션은 잘 작동하고 아주 좋네요ㅎㅎ 확인해보니까 비활성화됐을때도 포인터 커서라서 그 부분만 수정되면 좋을 거 같습니당
😎 담당자
식사 메뉴 고민중인 서향
ㅤ
🧩 이슈 번호
159
ㅤ
✨ 작업 내용
그룹 목록 페이지 구현 그룹 목록, 멤버 수가 많을 경우 페이지네이션 적용되도록 구현 그룹 전체/목록별 진척도 구분
ㅤ
💡 이번 pr을 올리기 전에 받은 피드백이 있나요? (있으면 체크리스트 작성)
📸 스크린샷 또는 GIF
ㅤ
✍🏻 참고사항
그룹 목록별 진척도는 아직 스타일 적용이 제대로 되지 않았는데, 늦어지기 전에 진행된 부분들 먼저 올려둡니다!!
ㅤ
코드 리뷰 남길때 참고용 (pr 작성할 때 지우지 말기)
P1: 꼭 반영해주세요 (Request changes) - 리뷰어는 PR의 내용이 서비스에 중대한 오류를 발생할 수 있는 가능성을 잠재하고 있는 등 중대한 코드 수정이 반드시 필요하다고 판단되는 경우, P1 태그를 통해 리뷰 요청자에게 수정을 요청합니다. 리뷰 요청자는 p1 태그에 대해 리뷰어의 요청을 반영하거나, 반영할 수 없는 합리적인 의견을 통해 리뷰어를 설득할 수 있어야 합니다.
P2: 적극적으로 고려해주세요 (Request changes) - P2에 대해 수용하거나 만약 수용할 수 없는 상황이라면 적합한 의견을 들어 토론할 것을 권장합니다.
P3: 웬만하면 반영해 주세요 (Comment) - P3에 대해 수용하거나 만약 수용할 수 없는 상황이라면 반영할 수 없는 이유를 들어 설명하거나 다음에 반영할 계획을 명시적으로 표현합니다.
P4: 반영해도 좋고 넘어가도 좋습니다 (Approve) - 해당 의견을 반영하는 게 좋을지 고민해 보는 정도면 충분합니다
P5: 그냥 사소한 의견입니다 (Approve) - 아무런 의견을 달지 않고 무시해도 괜찮습니다
PQ: 질문입니다 (Question) - 질문에는 꼭 답을 해주세요