[FE] 입금 상태를 변경하고 '수정완료' 버튼 클릭 시, 중복 이름 버그 발생 #611
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
jinhokim98
requested changes
Sep 25, 2024
pakxe
approved these changes
Sep 25, 2024
| @@ -20,7 +20,7 @@ const useRequestPutMembers = () => { | |||
| }, | |||
| }); | |||
|
|
|||
| return {putMember: mutate, ...rest}; | |||
| return {putMember: mutateAsync, ...rest}; | |||
Todari
approved these changes
Sep 25, 2024
| @@ -20,7 +20,7 @@ const useRequestDeleteMember = () => { | |||
| }, | |||
| }); | |||
|
|
|||
| return {deleteMember: mutate, ...rest}; | |||
| return {deleteMember: mutateAsync, ...rest}; | |||
There was a problem hiding this comment.
우리가 지금 사용하고 있는 react-query hook들이 다 같은 이름 형식으로 내보내고 잇는데,
deleteMember가 mutateAsync를 내보낸 거라는걸 알기 위해서는
deteleMemberAsync와 같은 식으로 내보내줘도 좋을 것 같아요~!
jinhokim98
approved these changes
Sep 26, 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.
issue
구현 사항
🚨 백엔드 api 수정 이후, merge 진행
버그 발생 상황
입금 상태만을 변경하고 '수정완료' 버튼을 클릭하면 "중복 이름입니다" 버그가 발생했습니다.
이에 대한 문제는 api 구조의 문제로 결론이 났습니다. 따라서 해당 문제의 해결은 백엔드의 수정 후에 정확히 확인해봐야 할 것 같습니다.
또 다른 버그 발생 상황 사전 방지
전체 참여자 delete,put 요청 mutate대신 mutateAsync를 사용하여 요청 순서 보장했습니다.
순서를 보장하지 않는다면 delete 요청 전에 put 요청이 먼저 갈 수 있다는 문제가 존재합니다.
delete 요청 전, put 요청이 가는 것이 왜 문제냐? 생각하실 수 있습니다. put 요청은 delete가 요청이 된 것을 기반으로 데이터가 존재합니다. 따라서 delete 요청이 무조건 선행 되어야 put 요청을 전송할 수 있습니다.
🫡 참고사항