Skip to content
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

[Fix] 인증 실패 메시지 뜨는 문제 수정 #508

Merged
merged 1 commit into from
Feb 10, 2024

Conversation

sumi-0011
Copy link
Member

🤔 해결하려는 문제가 무엇인가요?

closed #507

  • 미션 인증 submit 버튼을 여러번 클릭 시 API 중복 호출로 인해 에러 토스트가 뜨는 문제가 발생
  • 기존의 대처 (isPending)으로 되지 않은 이유는, API 호출 후에 라우팅이 되기 전 사이에 버튼 클릭이 일어나서 발생한 것으로 추정
  • sumit후 라우팅이 될 때 까지 loading으로 기다림,
  • 만약 오류가 발생했으면 loading을 풀어서 다른 동작을 수행하도록 함

Copy link

vercel bot commented Feb 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
10mm-client-web ✅ Ready (Inspect) Visit Preview Feb 9, 2024 9:47am

Copy link
Contributor

github-actions bot commented Feb 9, 2024

Copy link
Collaborator

@wade3420 wade3420 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

굳이 굳이 뮤테이션 상태를 써야한다!는 아니지만 sumit 성공 후 -> 라우트 사이의 텀이 문제라면 isSuccess 일때도 button을 막으면 되지 않을까 생각했지만 오히려 뮤테이션 두개쓰는 것보다 더 코드가 깔끔해져서 어프룹합니다 ( 훅 분리하면 좋을 것 같다는 생각은 드네요 )

@wade3420 wade3420 merged commit 99e363e into develop Feb 10, 2024
5 checks passed
@wade3420 wade3420 deleted the feat/mission-record-fail branch February 10, 2024 07:51
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] 미션 인증 submit 버튼 클릭시 에러 토스트 노출
2 participants