[TIL] 18주차 월요일. UT 피드백 점검, 단체 코드리뷰 시간

Minji Kim·2024년 8월 12일

내배캠TIL

목록 보기
72/73

UT 피드백 점검

김민지 티켓 검토 결과

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

얻게 된 것

  • 위의 내용 말고도 내 담당이 아닌 부분들까지 굉장히 많은 다양한 피드백을 받았는데, 아예 생각하지 못 한 부분들도 꽤 있어서 도움이 많이 되었다.
  • UT를 진행하고 받은 모든 피드백을 다 반영하지는 못 하니, 이렇게 정리해서 우리의 기획, 현재 상황, 그리고 앞으로의 일정과 계획에 따라 조정하는 작업을 잘 한 것 같다. 부팀장님이 피드백을 모두 정리해주시고 형식도 정해주셔서 따라가기만 했는데... 팀플 하면서 이렇게 계속 팀원 분들께 배우게 된다!
  • 하나 메모하면 좋을 것: 어떤 기기에서 사용했는지도 설문에 받으면 좋을 것 같다. 운영 체제뿐만 아니라 모바일의 경우 어떤 모바일 디바이스인지..

코드리뷰

그동안 빠르게 approve, merge 하느라 PR마다 코드리뷰를 거의 안 달고 확인 댓글만 짧게 남겼었는데, 한 번 쭉 다같이 코드리뷰를 해보자는 팀장님&부팀장님의 의견에 따라 코드리뷰를 해 보았다.

얻게 된 것

  • 코드를 전체적으로 검토해보면서(기술적인 부분은 거의 그냥 넘겼지만..) 내가 쓰지 않은 부분까지 프로젝트 전반적인 구조를 더 이해할 수 있게 되었다! 좋은 작업인 것 같다.
  • 놓치고 있던 자잘한 컨벤션도 다시 짚고 넘어갈 수 있게 된 시간이었다. 프로젝트 극 초반에 써놓고 수정하지 않았던 코드를 다시 보니까 컨벤션이 안 지켜져 있다는 걸 확인할 수 있었다.
  • 내 코드 리팩토링이 시급하다 (어렵다..)
  • 기술적인 면에서 나보다 실력이 좋은 팀원 분들의 조언을 얻을 수 있어서 유익했다.

0개의 댓글