-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
✨ 피드 신고하기 API #66
✨ 피드 신고하기 API #66
Conversation
This pull request is automatically being deployed by Amplify Hosting (learn more). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고 많으셨습니다.
mutate의 비동기 처리에 대해서 찾아봤는데, mutate 대신 mutateAsync를 사용하면 promise를 반환받을 수 있다고 합니다. 이 부분 참고하시면 도움이 될 것 같습니다...!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다. 병합하셔도 좋습니다!
@Legitgoons 오늘도 가르침에 감사합니다 선생님 👍 |
작업 이유
작업 사항
1️⃣ 피드 신고하기 API
2️⃣ 신고 카테고리
기존에 신고 카테고리를 아래 이유로 map에서 array로 변경하였습니다.
map.get(1)
,array[1]
=> O(1))3️⃣ ESLint 설정
리뷰어가 중점적으로 확인해야 하는 부분
신고하기 제출 이후에 서버에 정상 반영 된 이후에 모달창을 닫도록 구현하였는데, mutation이 비동기로 잡히지 않아서
await
가 정상적으로 적용되고 있지 않는 문제가 발생하였습니다..!이 부분을 어떻게 처리하면 좋을까요?
발견한 이슈