-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: match3 로직 추가 완료 #470
The head ref may contain hidden characters: "469-feat-match3-\uB85C\uC9C1-\uAC08\uC544\uC5CE\uAE30"
Conversation
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.
match3를 기준으로 분기 처리하신 것 확인했습니다. 고생 많으셨어요.
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.
코드 너무 잘 읽히네용! 전달되는 값들 차이 문제만 없다면 잘 될 것 같습니다 고생하셨어용~~~!
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.
match3로직 구현 잘 봤습니다 수고 많으셨습니다!
frontend/src/pages/match-up/page.js
Outdated
@@ -26,7 +21,7 @@ export default function Matchup($container, info = null) { | |||
} else { | |||
console.error("No match data available"); | |||
} | |||
setCards(data.data.data.match3); | |||
setCards(info.data.data.match3); |
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.
여기서 info였군요 고쳐주셔서 감사합니다!
✅ 풀_리퀘스트 체크리스트
🔄 변경 사항
match3 페이지에서 소켓 메시지 받아서 동적렌더링 해 주고 싶었는데 어려워서 못하겠습니다..
그냥 online-game 페이지에서 기다리다가 같이 matchup으로 넘겼어요
closes: #469