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

[Feature/Room] Implement random profile generation #53

Merged
merged 13 commits into from
Mar 8, 2025

Conversation

yunyoung1819
Copy link
Collaborator

#️⃣ 연관된 이슈

📝 작업 내용

  • 랜덤 프로필 이미지 생성
  • 랜덤 닉네임 생성

📸 스크린샷 (선택)

💬 리뷰 요구사항(선택)

@yunyoung1819 yunyoung1819 added the ✨ Feature new feature label Mar 6, 2025
@yunyoung1819 yunyoung1819 self-assigned this Mar 6, 2025
…ture/random-profile

# Conflicts:
#	kok-api/src/main/java/com/kok/kokapi/room/application/service/RandomProfileService.java
#	kok-api/src/main/java/com/kok/kokapi/room/application/service/RoomParticipantService.java
@yunyoung1819 yunyoung1819 marked this pull request as ready for review March 8, 2025 07:35
@yunyoung1819
Copy link
Collaborator Author

편하실때 리뷰 부탁드려요~!

@yunyoung1819 yunyoung1819 added the 🥇 Urgency 1 긴급도 높음 label Mar 8, 2025
Copy link
Member

@linirini linirini left a comment

Choose a reason for hiding this comment

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

LGTM! 소소한 리뷰 조금씩 남겨보았습니다!
수고하셨어용><

private final ObjectMapper objectMapper;

@Override
public void joinRoom(String roomId, Member member) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 지점에서 memberId 를 리턴해주는것이 좋을것 같다는 의견입니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

우선 여기에서 memberId를 리턴하는 방식으로 수정했어요

@yunyoung1819
Copy link
Collaborator Author

다 수정했어요

@yunyoung1819 yunyoung1819 merged commit 0938854 into develop Mar 8, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature new feature 🥇 Urgency 1 긴급도 높음
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants