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

[refactor] 등록자 프로필 조회 API 클래스 분리 #143

Merged
merged 1 commit into from
Jan 19, 2025

Conversation

2hyunjinn
Copy link
Collaborator

@2hyunjinn 2hyunjinn commented Jan 19, 2025

PULL REQUEST

#️⃣관련 이슈

🎋 작업중인 브랜치

💡 작업내용

  • 등록자 프로필 조회 API 클래스 분리

🔑 주요 변경사항

  • 등록자 프로필 조회 API 클래스 분리
public UserInfo getGroupUserInfo(GroupType groupType, Long groupId) {
        GroupUserInfoStrategy groupUserInfoStrategy = groupUserInfoStrategyRegistry.getGroupUserInfo(groupType);

        return UserInfo.of(userService.getUserById(groupUserInfoStrategy.getGroupUserInfo(groupId)));
    }

🏞 스크린샷

image

closes #140

@2hyunjinn 2hyunjinn added the 🏭 refactor refactoring code label Jan 19, 2025
@2hyunjinn 2hyunjinn requested a review from khyojun January 19, 2025 19:32
@2hyunjinn 2hyunjinn self-assigned this Jan 19, 2025
Copy link

Test Results

13 files  13 suites   3s ⏱️
59 tests 59 ✅ 0 💤 0 ❌
63 runs  63 ✅ 0 💤 0 ❌

Results for commit 7393cc0.

Copy link
Member

@khyojun khyojun left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 👍

@khyojun khyojun merged commit b682e72 into main Jan 19, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏭 refactor refactoring code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[refactor] 등록자 프로필 조회 API 클래스 분리
2 participants