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

nickname 표시, 버그수정, 마이페이지 로직 #109

Merged
merged 12 commits into from
Nov 30, 2022

Conversation

JJongBin
Copy link
Collaborator

📕 제목

PR 제목

📗 작업 내용

구현 내용 및 작업 했던 내역

  • phaser nickname 표시
  • 전역상태 소켓이 연결되기 전 동기화되도록 수정
  • 닉네임 변경 에러메세지 및 변경 반영
  • 캐릭터(hair) 변경
  • 초기 접속, 다른 유저 접속 소켓 이벤트(on) 분리
  • 서버에서 받아오는 데이터의 공백 제거

📘 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • 다른 유저가 변경한 캐릭터 및 닉네임 반영 x (새로 유저 목록 도는 변경한 유저의 정보를 받아오는 로직 필요?)
  • 소켓이 연결되고 이벤트를 받아오는 부분에서 이벤트를 감지하는 로직의 순서가 불분명(userCreated를 on시키기전에 이벤트가 발생하지 않는가?)

@JJongBin JJongBin added 프론트엔드 프론트엔드 버그 리팩토링 Refactoring 기능 기능 추가 labels Nov 30, 2022
@JJongBin JJongBin self-assigned this Nov 30, 2022
@hj1n
Copy link
Collaborator

hj1n commented Nov 30, 2022

클라이언트와 서버 소켓 연결이 진행되고 있네요 ㅎㅎ 좋아요!!

@hj1n hj1n merged commit 9f9080d into boostcampwm-2022:develop Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 기능 추가 리팩토링 Refactoring 버그 프론트엔드 프론트엔드
Projects
Status: 🎄 Done
Development

Successfully merging this pull request may close these issues.

2 participants