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

[Feat] 각종 버그 수정 및 로그인시 클라측에서 id받는 로직 추가 #196

Merged
merged 7 commits into from
Nov 28, 2024

Conversation

mssak
Copy link
Collaborator

@mssak mssak commented Nov 28, 2024

개요

Resolves: #194

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

작업 내용

  • 게스트 로그인시 클라측에서 파라미터로 지정받은 id를 기반으로 생성할 수 있게 하였습니다.
    • 로그인 id 세션을 저장하여 혹시 중복된다면 지정된 id 뒤에 숫자를 늘려서 반환합니다.
  • gsid를 4자리로 줄였습니다.
    • 중복이 안나올때까지 랜덤으로 uuid를 계속 돌리는 방식입니다.
  • 게임 도중에 방에 못들어오게 수정했습니다.
    • 게임 도중 방에 들어오려고 하면 error 이벤트를 클라에 반송합니다.
  • 어색한 단어들을 삭제하고 가구, 감정 테마를 추가했습니다.
  • 최대 방 참가 인원수가 7명이던 버그 수정

스크린샷

  • gsid
    image

  • 최대 방 참가 제한
    image

  • 화면은 7명이상 들어와짐 (수정해야 함)
    image

  • 게임 도중 못들어옴
    image

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고 (Ctrl + 클릭하세요.)
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

@mssak mssak self-assigned this Nov 28, 2024
@zizonyoungjun zizonyoungjun merged commit 09bd7e8 into develop Nov 28, 2024
4 checks passed
@zizonyoungjun zizonyoungjun deleted the feature/#192 branch November 28, 2024 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

사용자는 게스트 로그인 시에 닉네임을 설정할 수 있다
2 participants