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.

Sign up for GitHub

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

Jump to bottom

[BE] 회원가입시 기본 닉네임 설정 #224

Merged
merged 4 commits into from
Nov 28, 2024
Merged

[BE] 회원가입시 기본 닉네임 설정 #224

merged 4 commits into from
Nov 28, 2024

Conversation

uuuo3o
Copy link
Collaborator

@uuuo3o uuuo3o commented Nov 27, 2024

✅ 주요 작업

  • 회원가입시 자동으로 '익명의 투자자' + userId 번호로 닉네임 설정
  • 닉네임 변경할 때 기본 닉네임인 '익명의 투자자'가 포함된 닉네임으로 변경되지 않게 로직 추가

💭 고민과 해결과정

  • 사실 제 독단으로 추가한 부분이라 내일 이야기해보긴 해야 합니다..ㅎㅎ
  • 나중에 정규식이나 사용 불가능한 단어(게임에서 필터링되는 단어같은..!)도 닉네임 설정 때 못쓰게 해야 할 거 같긴 합니다.

@uuuo3o uuuo3o added BE 백엔드 API API 구현 labels Nov 27, 2024
@uuuo3o uuuo3o requested review from jinddings and sieunie November 27, 2024 16:55
@uuuo3o uuuo3o self-assigned this Nov 27, 2024
jinddings
jinddings approved these changes Nov 28, 2024
View reviewed changes
Copy link
Collaborator

@jinddings jinddings left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

sieunie
sieunie approved these changes Nov 28, 2024
View reviewed changes
Copy link
Collaborator

@sieunie sieunie left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~~

@uuuo3o uuuo3o merged commit 5a044e4 into back/main Nov 28, 2024
2 checks passed
@uuuo3o uuuo3o deleted the feature/api/defaultUserName branch December 5, 2024 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jinddings jinddings jinddings approved these changes

@sieunie sieunie sieunie approved these changes

Assignees

@uuuo3o uuuo3o

Labels
API API 구현 BE 백엔드
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@uuuo3o @jinddings @sieunie