김민지 티켓 검토 결과
- [제안] 본인이 입력한 채팅을 마이페이지에서 확인 가능하도록
-> 채팅 탭이 별도로 존재하기 때문에 구현하지 않아도 될 것 같다.
-> 지금은 참여한 여정의 상세페이지에서 '참여하기' 누르면 '이미 참여한 여정입니다?'라고 뜨는데 대신 '채팅하기' 버튼으로 바꾸는 방법?- [버그] 채팅 탭을 누르면 전 페이지로 돌아감
-> trips 페이지에서 참여한 여정이 없는 상태에서 chat 페이지로 넘어가신 거로 추정.
-> 채팅이 없으면 "아직 참여한 여정이 없습니다! 트립 버디즈와 함께 즐거운 여행을 시작해 볼까요?" 문구 뜨고 클릭하면 trips로 이동하는 로직으로 변경하는 건 어떤지. - 12일까지- [제안][공통] 랜덤 채팅방
-> 구현하기 어려울 것 같습니다.- [제안] 채팅 메시지 읽음 기능, 메시지 삭제 기능
-> 읽음 기능 추가 예정 - 14일까지
-> 삭제 기능은 후순위로 (피그마에도 없음)- [제안] 여정 상세에서 글을 보다가 뒤로 가려면 불편함 -> 여정 상세에서 헤더 고정
-> 모바일에선 헤더를 탭메뉴와 마찬가지로 fixed로 상시 고정하는 방법 어떠신지 팀원분들 의견 구하기- [제안] 하단 탭 메뉴 스크롤을 살짝 해야 보여서 불편 -> 고정
-> home 페이지에서만 스크롤 해야 보이는 거로 확인. 수정해보겠습니다.- [미구현] 여정 리스트 모바일 헤더 notification 아이콘 미작동
-> notification 리스트 페이지로 이동하도록 구현 완료. 알림 기능은 개발 중. - 14일까지- [제안] 홈 페이지 ‘전체보기’가 버튼인지 모르겠음 (와이어프레임과 상이하네요)
-> 바로 수정하겠습니다 - 12일까지- [제안] 여정 출발 전 휴대폰 알림
-> 후순위로 두기- [제안] 홈 헤더 “예정된 ~여정이 ~일 남았어요” 클릭 시 해당 여정 상세로 리다이렉트
-> 이 부분 생각했던 건데 좋은 피드백입니다. 구현 하겠습니다. - 12일까지
그동안 빠르게 approve, merge 하느라 PR마다 코드리뷰를 거의 안 달고 확인 댓글만 짧게 남겼었는데, 한 번 쭉 다같이 코드리뷰를 해보자는 팀장님&부팀장님의 의견에 따라 코드리뷰를 해 보았다.