#156 택시 탑승이 혼자인 경우, 자동으로 결제 완료 및 과거 참여 방으로 이동

[Hyogyeong8]

Summary

It closes #156

Further Work

• searchByUserHandler

[Hyogyeong8]

searchByUserHandler말고 또 고쳐야 할 함수가 있을까요?
그리고 정산 완료로 바꾸고, 과거 참여방으로 이동시키는 것 외에 채팅을 닫는다던가 하는 작업들은 필요 없나요?

[14KGun]

@imYourChoi 이거 back에서 처리하면 front에서 내 방 정보 수정되었는지 감지할 수 있는 로직이 있을까요??

(생각난 해결 방안으로는, front에서 택시 출발시각 지났고 혼자 참여 중이면 자동으로 back에 정산하기 api 호출하는 방안이 있기는 합니다.)

[imYourChoi]

@14KGun 지금 생각나는 거는 그 작업을 Skeleton에서 path 바뀔때마다 해주는 수 밖에는 없는것 같은데 비효율적인것 같기는 하네요

[Hyogyeong8]

지금 상태로는 내 방 확인을 해야만 이 기능이 구현됩니다. 좀 더 정확한 구현을 위해서는 rooms/join API 안에서도 한 번 더 이 기능을 구현해야 할 것 같습니다. (현재 참여 중인 방이 5개 이상이라고 오류를 보내기 전에) 하지만 그렇게 하면 코드가 길어질 것 같아서 넣을지 고민하고 있습니다.

[14KGun]

지금 상태로는 내 방 확인을 해야만 이 기능이 구현됩니다. 좀 더 정확한 구현을 위해서는 rooms/join API 안에서도 한 번 더 이 기능을 구현해야 할 것 같습니다. (현재 참여 중인 방이 5개 이상이라고 오류를 보내기 전에) 하지만 그렇게 하면 코드가 길어질 것 같아서 넣을지 고민하고 있습니다.

네네, 수고하셨습니다만.... 😢 front에서 내가 정산이 자동으로 되었는지 확인할 수 있는 로직이 없어. 아직 머지되면 않될 듯 합니다 !!

@Hyogyeong8 , @imYourChoi

[Hyogyeong8]

지금 상태로는 내 방 확인을 해야만 이 기능이 구현됩니다. 좀 더 정확한 구현을 위해서는 rooms/join API 안에서도 한 번 더 이 기능을 구현해야 할 것 같습니다. (현재 참여 중인 방이 5개 이상이라고 오류를 보내기 전에) 하지만 그렇게 하면 코드가 길어질 것 같아서 넣을지 고민하고 있습니다.

네네, 수고하셨습니다만.... 😢 front에서 내가 정산이 자동으로 되었는지 확인할 수 있는 로직이 없어. 아직 머지되면 않될 듯 합니다 !!

@Hyogyeong8 , @imYourChoi

죄송하지만 이해가 잘 안 갑니다ㅠㅠ front에서 정산이 되었는지를 확인해야 하나요? back에서 정산 완료시킨 방을 반환하면 되는 게 아닌가요? 로직이 완전하지 않아서 머지 안 되어도 저는 상관 없습니다:)

[14KGun]

지금 상태로는 내 방 확인을 해야만 이 기능이 구현됩니다. 좀 더 정확한 구현을 위해서는 rooms/join API 안에서도 한 번 더 이 기능을 구현해야 할 것 같습니다. (현재 참여 중인 방이 5개 이상이라고 오류를 보내기 전에) 하지만 그렇게 하면 코드가 길어질 것 같아서 넣을지 고민하고 있습니다.

네네, 수고하셨습니다만.... 😢 front에서 내가 정산이 자동으로 되었는지 확인할 수 있는 로직이 없어. 아직 머지되면 않될 듯 합니다 !!

죄송하지만 이해가 잘 안 갑니다ㅠㅠ front에서 정산이 되었는지를 확인해야 하나요? back에서 정산 완료시킨 방을 반환하면 되는 게 아닌가요? 로직이 완전하지 않아서 머지 안 되어도 저는 상관 없습니다:)

front 에서 roomlist를 확인 하는 API를 최대한 효율적으로 호출할려고 매번 저 API를 호출하지 않습니다. ㅜ
자신이 방에 참가하였을 때, 정산/결제 요청을 하였거나 방을 나갔을 때 만 roomlist API를 호출하여 내 방 정보를 update 하고 있습니다.

근데, 저 세 조건에 만족하지 않았는데 back에서 내 방 정보를 update한다면, front 에서 가지고 있는 내 방 정보랑 back의 db에 저장되어 있는 내용이랑 conflict이 날 것 같습니다.

@hyogyeong