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

[GGFE-188] api 응답 로딩 처리 #991

Merged
merged 29 commits into from
Sep 11, 2023
Merged

Conversation

yoouyeon
Copy link
Member

@yoouyeon yoouyeon commented Sep 8, 2023

📌 개요

  • 모달에 있는 버튼들을 ModalButton 으로 수정했습니다.

💻 작업사항

  • 모달이 많네요... 변경된 파일이 많긴 한데 변경 내용은 대부분 비슷합니다..ㅎㅎ
  • api 로딩 처리가 필요한 부분에는 isLoading state로 로딩 여부 확인을 하도록 했습니다!
  • 카카오 연동 모달의 경우에는 다른 모달 버튼들과 스타일이 많이 달라서 공통 모달 버튼을 적용하지 못했습니다 ^_ㅜ
  • 모달들 한번씩 들어가보셔서 로딩 처리 잘 되는지 확인해주시면 감사하겠습니다!!!!!! 😇

✅ 변경로직

@yoouyeon yoouyeon self-assigned this Sep 8, 2023
@yoouyeon
Copy link
Member Author

yoouyeon commented Sep 8, 2023

다시 보니 게임 결과 입력 부분에 기존 코드를 많이 유지하면서 로딩 처리를 할 수 있을 수 있을 것 같아서 시도해보고 다시 PR 열겠습니다!

@yoouyeon yoouyeon closed this Sep 8, 2023
@yoouyeon yoouyeon reopened this Sep 8, 2023
Copy link
Contributor

@PHJoon PHJoon left a comment

Choose a reason for hiding this comment

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

모달들 다 잘 작동하네요! 모달버튼 컴포넌트 통일되니까 보기 훨씬 좋네요 👍 양이 엄청 많은데 고생하셨습니다!! 👍

Copy link
Member

@hyobb109 hyobb109 left a comment

Choose a reason for hiding this comment

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

모달 정말 많은데.. 다 적용하시느라 정말 수고 많으셨습니다! 출석하기는 테스트 못해봤지만 잘 될 것 같아요!! 고생 많으셨습니다!(●'◡'●)

@yoouyeon yoouyeon merged commit 3621263 into main Sep 11, 2023
1 check passed
@yoouyeon yoouyeon deleted the GGFE-188-api-응답-로딩-처리 branch September 11, 2023 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants