netlify[bot]
commented
1 month ago Deploy request for coworkers-1 pending review.
Visit the deploys page to approve it
| Name | Link |
|---|---|
| Latest commit | 729df5dab9522b9026141f593bdcf588bf5c3c27 |
Closed kanghyosung1 closed 1 month ago
netlify[bot]
commented
1 month ago Visit the deploys page to approve it
| Name | Link |
|---|---|
| Latest commit | 729df5dab9522b9026141f593bdcf588bf5c3c27 |
😎 담당자
빌드업 하는 강효성ㅤ
🧩 이슈 번호
ㅤ-#120
✨ 작업 내용
ㅤ
💡 이번 pr을 올리기 전에 받은 피드백이 있나요? (있으면 체크리스트 작성)
[x] Button color='red'일 떄 hover, active 효과 적용
[x] 게시글에서 댓글 달때 로딩중일때 '수정중입니다', '삭제중입니다 ' 삭제
[x] div 버튼들 버튼 테그로 수정
[x] pickerButton 이전 상태로 수정 (상태값 지워진 이슈) https://github.com/part4-1st-team/coworkers/pull/112#discussion_r1770761499
[x] ProfileImage.tsx 파일로 교체
[x] 좋아요 버튼 9999+ 처리
[x] 파일명 오타 수정
[x] useInview 를 활용한 무한 스크롤 수정
[x] useForm 일부 적용
[ ] 수정 안 한 피드백
[ ] 베스트게시글 반응형 구현
구현중 반응형에따라 개수가 변화되지 않아 해결하는데 시간이 더 필요 함 https://github.com/part4-1st-team/coworkers/pull/114#discussion_r1770775547
[ ] useForm 일부분 적용시 사이트 이펙트 발생하여 차후 리펙토링 시간에 작업 예정
ㅤ
📸 스크린샷 또는 GIF
ㅤ
✍🏻 참고사항
ㅤ
코드 리뷰 남길때 참고용 (pr 작성할 때 지우지 말기)
P1: 꼭 반영해주세요 (Request changes) - 리뷰어는 PR의 내용이 서비스에 중대한 오류를 발생할 수 있는 가능성을 잠재하고 있는 등 중대한 코드 수정이 반드시 필요하다고 판단되는 경우, P1 태그를 통해 리뷰 요청자에게 수정을 요청합니다. 리뷰 요청자는 p1 태그에 대해 리뷰어의 요청을 반영하거나, 반영할 수 없는 합리적인 의견을 통해 리뷰어를 설득할 수 있어야 합니다.
P2: 적극적으로 고려해주세요 (Request changes) - P2에 대해 수용하거나 만약 수용할 수 없는 상황이라면 적합한 의견을 들어 토론할 것을 권장합니다.
P3: 웬만하면 반영해 주세요 (Comment) - P3에 대해 수용하거나 만약 수용할 수 없는 상황이라면 반영할 수 없는 이유를 들어 설명하거나 다음에 반영할 계획을 명시적으로 표현합니다.
P4: 반영해도 좋고 넘어가도 좋습니다 (Approve) - 해당 의견을 반영하는 게 좋을지 고민해 보는 정도면 충분합니다
P5: 그냥 사소한 의견입니다 (Approve) - 아무런 의견을 달지 않고 무시해도 괜찮습니다
PQ: 질문입니다 (Question) - 질문에는 꼭 답을 해주세요