Skip to content

Comments

[fix] 동아리 수정 시 발생하는 필드 누락 문제 해결#212

Merged
Due-IT merged 1 commit intodevelop/befrom
fix/#211-club-update-field-bug
Mar 18, 2025
Merged

[fix] 동아리 수정 시 발생하는 필드 누락 문제 해결#212
Due-IT merged 1 commit intodevelop/befrom
fix/#211-club-update-field-bug

Conversation

@Zepelown
Copy link
Member

@Zepelown Zepelown commented Mar 18, 2025

#️⃣연관된 이슈

#211

📝작업 내용

update 메소드 사용 시 update와 관련없는 필드가 사라지는 문제

    public void update(ClubInfoRequest clubInfoRequest){
        this.recruitmentEnd = clubInfoRequest.recruitmentEnd();
        this.recruitmentStart = clubInfoRequest.recruitmentStart();
        this.presidentName = clubInfoRequest.clubPresidentName();
        this.presidentTelephoneNumber = clubInfoRequest.telephoneNumber();
        this.recruitmentTarget = clubInfoRequest.recruitmentTarget();
        this.introduction = clubInfoRequest.introduction();
        this.tags = clubInfoRequest.tags();
    }

-> 기존 빌더 패턴에서 update 메소드를 추가해 변경사항만 수정하도록 반영

@github-actions
Copy link

Test Results

1 tests   1 ✅  0s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit 5ea9ee9.

Copy link
Collaborator

@PororoAndFriends PororoAndFriends left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

Copy link
Collaborator

@Due-IT Due-IT left a comment

Choose a reason for hiding this comment

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

고생하셨심당

@Due-IT Due-IT merged commit caa323b into develop/be Mar 18, 2025
2 checks passed
@Zepelown Zepelown deleted the fix/#211-club-update-field-bug branch October 12, 2025 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants