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

[FE - #109 ] Quiz-Session 소켓 연결 및 리팩토링 #110

Merged
merged 23 commits into from
Nov 24, 2024

Conversation

chan-byeong
Copy link
Collaborator

@chan-byeong chan-byeong commented Nov 23, 2024

#️⃣연관된 이슈

[#109] QUIZ 관련 소켓 연결하기

📝작업 내용

  • Quiz와 연관된 소켓 이벤트들을 연결하였습니다.
  • Background를 이동하여 렌더링을 최적화 하였습니다.

Quiz Background 컴포넌트 index.tsx -> QuizBox.tsx
유저가 이모지를 클릭하거 이모지 이벤트를 받을 경우 index.tsx에 존재하는 reaction State로 인해 퀴즈 세션 페이지 전체가 렌더링되는 이슈가 있었습니다.

reactionState를 QuizBox 컴포넌트로 이동하고 Quiz Background 컴포넌트도 QuizBox 컴포넌트로 이동하여 전체 페이지가 렌더링되지 않도록 수정하였습니다.

스크린샷

💬리뷰 요구사항

참고 자료

@chan-byeong chan-byeong added ✨ Feature 기능 작업 🤖 Refactor 코드 개선, 가독성 개선, 주석 labels Nov 23, 2024
Copy link
Collaborator

@dooohun dooohun left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~

quiz: QuizData;
tick: { currentTime: number; elapsedTime: number; remainingTime: number };
Copy link
Collaborator

Choose a reason for hiding this comment

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

요 친구도 따로 빼도 좋을 거 같아요!

@chan-byeong chan-byeong merged commit 72755f3 into develop Nov 24, 2024
@dooohun dooohun deleted the feature/FE/quiz-session-socket branch November 28, 2024 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 작업 🤖 Refactor 코드 개선, 가독성 개선, 주석
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants