search

sparcs-kaist / taxi-front

KAIST Taxi Party Matching Web Service
https://taxi.kaist.ac.kr
MIT License
14 stars 2 forks source link

#778 홈 화면 방 팝업에서 참여 중인 방 선택 시 바로가기 버튼 추가 #779

Closed jinhyeonkwon closed 4 months ago

jinhyeonkwon commented 5 months ago

Summary

It closes #778

구현한 것

packages/web/src/components/ModalPopup/Body/BodyRoomSelection.tsx 를 수정하여, 1) 이미 참여 중인 방의 경우에도 활성화된 버튼이 나타나도록 하였으며 2) requestJoin 을 수정하여, 해당 버튼을 누르면 history.push를 사용하여 바로 방으로 이동하는 로직을 추가하였습니다.

리뷰 요청

requestJoin을 수정하면서, 이미 참여 중인 방은 API 호출이 필요 없기에 기존의 코드를 모두 무시하도록 하였습니다. requestJoin의 수정 사항이 적절한지를 중심으로 리뷰해주시면 감사하겠습니다.

논의 거리

  1. 홈에서 모달을 띄운 채로 다른 창에서 탑승 취소해버리면, 추후에 모달에서 버튼을 누르면 방 참여(또는 존재) 여부를 검증하지 않고 url을 이동하기에 무한 로딩이 뜹니다. 이것을 검증하기 위해 API 콜을 날리는 게 좋은지, 아니면 이런 극단적인 경우를 위해 API 콜을 낭비하지 않는 게 좋은지 논의하고 싶습니다.
  2. 11시 10분 출발 방은 11시 10분 59초까지 홈 화면에 보이는 것 같습니다. 이로 인해 홈 화면에 보이지만, isDepart로 인해 (참여 중인 경우의 바로가기 버튼도 포함) 버튼이 비활성화 되어 있는 상태가 1분간 지속됩니다. 이것이 새로운 이슈로서 논의되어야 할 것 같습니다. 우선은 이 PR에 대한 추가 커밋을 통해, 그 1분 동안 이미 참여한 방이라면 비활성화 버튼 대신 바로가기 버튼이 뜨도록 하는 코드를 올려두었습니다.

Images or Screenshots

수정 전

image

수정 후

image

피드백 반영 : 버튼 메시지 수정

image

Further Work

netlify[bot] commented 5 months ago

Deploy Preview for taxi-dev-preview ready!

Name Link
Latest commit 9239357f79bf35aa7ec14ec477262e6f187a504f
Latest deploy log https://app.netlify.com/sites/taxi-dev-preview/deploys/661378f6a451550008f5e32f
Deploy Preview https://deploy-preview-779--taxi-dev-preview.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

jinhyeonkwon commented 4 months ago

민찬님 피드백까지 반영했어요!