#784 송금하지 않은 방이 있는 경우 추가 방 참여 제한 #787
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jinhyeonkwon
requested review from
happycastle114,
ybmin,
jyjy1229 and
predict-woo
3 tasks
✅ Deploy Preview for taxi-dev-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
kmc7468
reviewed
May 15, 2024
| room.part.find((item: any) => item._id === loginInfo?.oid) | ||
| .isSettlement === "send-required" && room.isDeparted | ||
| ); // 다른 사람이 정산을 올렸으나 내가 아직 송금하지 않은 방이 있는지 여부 (추가 입장 제한에 사용) | ||
| // item : any 가 좋은 방법인지 모르겠습니다 |
There was a problem hiding this comment.
이번 PR에서 방 타입 정의가 같이 이루어져도 괜찮을 것 같네요!
There was a problem hiding this comment.
회의 논의 결과, 백엔드가 TS로 넘어가면 타입 공유를 해서 쓰는걸로 결정했습니다.
xMHW
reviewed
May 21, 2024
| @@ -68,6 +69,14 @@ const AddRoom = () => { | |||
| useState<boolean>(false); | |||
| //#endregion | |||
|
|
|||
| const myOngoingRoom = myRooms?.ongoing.slice() ?? []; // InfoSection의 sortedMyRoom에서 정렬만 뺐습니다. | |||
There was a problem hiding this comment.
useMemo 사용 or loginInfo recoilState 에 추가하는게 좋을 것 같습니다.
kmc7468
approved these changes
May 21, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
It closes #784
미정산 문제에 대한 대응 중 하나로, 다른 참여자가 정산을 올렸으나 본인이 송금하지 않은 방이 있다면 1) 추가 방 참여와 2) 새로운 방 생성을 제한합니다.
중점적으로 피드백 받고 싶은 부분
sortedMyRoom) 에서 혹시 정산 또는 결제 (이 PR은 정산이 진행 중이나 본인이 송금하지 않은 상황에 한정되므로 조건은 다릅니다) 가 이루어지지 않은 방이 있는지를 확인하는notOver라는 변수가 있습니다. 이 과정을 최대한 보존하면서 아래와 같이 수정하였는데, 관련 질문이 있습니다.sortedMyRoom대신 정렬만 뺀myOngoingRoom을 선언하여 사용했습니다. 이것이 적절한 사용인지 궁금합니다. 정렬을 하지 않을 것이라면 이렇게 선언하는 것이 불필요한 중복 변수 선언이 될 수 있는 여지가 있는지 궁금합니다.notOver대신, 정산 진행 중이나 본인이 송금하지 않은 방이 있는지 알기 위해서, 각 방에 대하여part안에서 본인 id와 일치하면서isSettlement === "send-required"인 방을 찾아 그 여부를notPaid라는 변수에 저장하는 로직이 들어가 있습니다. 방 참여 쪽 수정 사항이 들어간BodyRoomSelection에는loginInfo를 이미 가져오고 있어서 그대로 사용했으나, 방 생성 쪽 수정 사항이 들어간 Addroom의 index.tsx에서는 새롭게const loginInfo = useValueRecoilState("loginInfo");를 추가하였습니다. 문제가 되지 않을 것으로 판단했으나 적절한지 궁금합니다.notPaid를 설정하는 과정에서 any type으로 인해 VSCode에서 뜨는 에러를 없애기 위해 arrow function의 매개변수를item: any라는 형식으로 썼습니다. (이것을 명시하기 전, 오류가 뜨는 상황에서도 작동은 정상적으로 되었습니다) 이것이 적절한 방법이 아닌 것 같아서, 개선할 방법이 있는지 궁금합니다.사소한 부분이나, 비활성화된 버튼의 텍스트가 적절한지 궁금합니다.
Images or Screenshots
Further Work