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.

Sign up for GitHub

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

Jump to bottom

안내면진다 버튼 문제 해결 #667

Open
wants to merge 2 commits into
base: develop-frontend
Choose a base branch
from
Open

안내면진다 버튼 문제 해결 #667

wants to merge 2 commits into from
+74 −66

Conversation

cys4585
Copy link
Contributor

@cys4585 cys4585 commented Oct 16, 2024

PR의 목적이 무엇인가요?

안내면진다 버튼 문제 해결

이슈 ID는 무엇인가요?

설명

  • 안내면진다 결과 확인 따닥 문제 해결
    • useRef로 true false 값 관리
    • true면 disabled, false면 enabled 처리
    • button click해서 mutateAsync로 비동기 API 대기 및 ref를 true로 변경, mutateAsync 끝나면 ref를 false로 변경
    • mutate가 invalidate하는 query가 isFetching일 때도 disabled 처리 (새로운 데이터를 얻어오고 나서 enabled 처리)
  • 참여자 없을 때 버튼 비활성화

질문 혹은 공유 사항 (Optional)

@cys4585 cys4585 added FE 프론트엔드 관련 이슈입니다. 🚑 버그 fix (develop에서 파생된 문제) labels Oct 16, 2024
@cys4585 cys4585 self-assigned this Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ss0526100 ss0526100 Awaiting requested review from ss0526100

@jaeml06 jaeml06 Awaiting requested review from jaeml06

At least 2 approving reviews are required to merge this pull request.

Assignees

@cys4585 cys4585

Labels
FE 프론트엔드 관련 이슈입니다. 🚑 버그 fix (develop에서 파생된 문제)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cys4585