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: 소셜 로그인 시 임시 회원가입 할 때 랜덤 생성 닉네임 넣어주는 기능 구현 #212

Merged
merged 7 commits into from
Jan 24, 2024

Conversation

uwoobeat
Copy link
Member

@uwoobeat uwoobeat commented Jan 23, 2024

🌱 관련 이슈

📌 작업 내용 및 특이사항

  • 로그인 로직에 fetchOrCreate 두 번 호출하는 부분 있어서 제거 및 개선
  • 처음 회원가입하는 유저에게 임시로 동물 닉네임 생성하는 기능 구현 (ex. 행복한사자)
    • 동물 이름은 2~4자 사이, 총 길이는 7~8자 사이가 되도록 단어 선정
  • 현재 nickname과 profileUrl이 아무 의미없이 VO로 묶여있어서 추후 리팩토링 필요함

📝 참고사항

📚 기타

@uwoobeat uwoobeat requested review from kdomo and char-yb January 23, 2024 16:59
@uwoobeat uwoobeat self-assigned this Jan 23, 2024
Copy link

sonarqubecloud bot commented Jan 23, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

44 New issues
0 Security Hotspots
48.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Member

@kdomo kdomo left a comment

Choose a reason for hiding this comment

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

LGTM 전략패턴 굳굳

Copy link
Member

@char-yb char-yb left a comment

Choose a reason for hiding this comment

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

LGTM이용
닉네임 generate 메서드에 요청사항있는데 사소해서 고민해주시고, merge 진행하면 될 거 같아용

Copy link
Member

Choose a reason for hiding this comment

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

볼수록 너무 웃겨용😂

Comment on lines +21 to +23

return prefix + animalName;
}
Copy link
Member

Choose a reason for hiding this comment

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

혹시 prefix + " " + animalName 이렇게 가능할까용?
형용사 띄어쓰기 동물이름이 좋아보여서용

Copy link
Member Author

Choose a reason for hiding this comment

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

음 닉네임에는 공백 없는게 좋다고 생각해서요

@uwoobeat uwoobeat merged commit c3a3e87 into develop Jan 24, 2024
3 checks passed
@github-actions github-actions bot added the merged 머지된 PR label Jan 24, 2024
@uwoobeat uwoobeat deleted the feature/207-temp-nickname branch January 24, 2024 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged 머지된 PR
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

✨ 소셜 로그인 시 임시 회원가입 할 때 랜덤 생성 닉네임 넣어주는 기능 구현
3 participants