[Fix] 온보딩 v2 api 수정

[eraser502]

Related issue 🛠

• closed [Fix] 온보딩 v2 api 수정 #207

Work Description 📝

• 기존 온보딩 api에서 값이 저장되지 않던 오류 해결
  • 수정 메서드에서 파라미터로 받은 값을 저장하지 않고 있었던 부분을 수정했어요
  • 추가적으로 personalBackground 값 또한 저장하지 않고 있었기에 해당 부분도 추가하였어요.

ScreenShots 📷

To Reviewers 📢

Summary by CodeRabbit

릴리스 노트

• 새로운 기능
  • 회원 가입 및 프로필 업데이트 시 개인 배경 정보를 추가로 수집하고 관리할 수 있도록 개선되었습니다.

[coderabbitai]

📝 Walkthrough

개요

Member 엔티티의 updateInfo() 메서드에 personalBackground 매개변수를 추가하고, MemberService의 온보딩 메서드들에서 이 필드를 전달하도록 수정했습니다.

변경사항

응집 영역 / 파일	요약
Member 엔티티 업데이트 src/main/java/org/sopt/kareer/domain/member/entity/Member.java	updateInfo() 메서드 서명에 String personalBackground 매개변수를 추가하고, 메서드 본문에서 이 필드를 엔티티에 할당하도록 수정
MemberService 호출 수정 src/main/java/org/sopt/kareer/domain/member/service/MemberService.java	onboardMember()와 onboardMemberV2() 메서드에서 member.updateInfo() 호출 시 request.personalBackground() 인자를 추가

예상 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~10분

관련 가능성이 있는 PR

• [Feat] 온보딩 예외처리 주석 해제 #134: 동일한 Member.updateInfo() 로직을 수정하는 PR로, pending/onboard 어서션 처리와 관련된 변경이 있어 직접적인 관련성이 있습니다.

리뷰어 제안

• hyomee2
• jeong1112

시 🐰

개인의 배경을 담아,
온보딩 흐름이 더 풍성해졌네요 🌿
한 줄의 매개변수,
서비스 전체에 퍼지니,
변화의 물결 일렁이며 🌊

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Linked Issues check	❓ Inconclusive	연결된 이슈 #207에 구체적인 요구사항이나 수용 기준이 명시되어 있지 않아 평가할 수 없습니다.	이슈 #207에 명확한 요구사항과 검증 기준을 추가하여 PR이 모든 목표를 충족하는지 확인하시기 바랍니다.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 온보딩 v2 API 수정이라는 PR의 주요 변경사항을 명확하게 요약하고 있습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 온보딩 v2 API 수정과 personalBackground 저장 추가라는 명시된 목표와 관련이 있습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/#207-onboard

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

src/main/java/org/sopt/kareer/domain/member/entity/Member.java (1)

85-99: 메서드 파라미터가 많습니다 - 향후 리팩토링을 고려해주세요.

updateInfo 메서드가 15개의 파라미터를 받고 있어 가독성과 유지보수성이 떨어질 수 있습니다. 현재 버그 수정은 올바르게 되었으나, 향후 리팩토링 시 파라미터 객체(Parameter Object) 패턴이나 Builder 패턴 사용을 고려해주세요.

예시:

// 파라미터 객체 예시
public void updateInfo(MemberUpdateCommand command) {
    assertPendingStatus();
    this.name = command.name();
    // ...
}

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/main/java/org/sopt/kareer/domain/member/entity/Member.java` around lines
85 - 99, updateInfo currently accepts 15 parameters which harms readability and
maintainability; refactor by introducing a parameter object (e.g.,
MemberUpdateCommand or MemberUpdateDto) or a Builder to wrap those fields,
change the updateInfo signature to accept that single object (public void
updateInfo(MemberUpdateCommand command)), keep the existing precondition call
(assertPendingStatus()) and replace direct parameter usages with command
accessors (e.g., this.name = command.name(), this.birthDate =
command.birthDate(), etc.), and update all call sites to construct and pass the
new command/builder; this preserves behavior while making the method concise and
easier to evolve.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@src/main/java/org/sopt/kareer/domain/member/entity/Member.java`:
- Around line 85-99: updateInfo currently accepts 15 parameters which harms
readability and maintainability; refactor by introducing a parameter object
(e.g., MemberUpdateCommand or MemberUpdateDto) or a Builder to wrap those
fields, change the updateInfo signature to accept that single object (public
void updateInfo(MemberUpdateCommand command)), keep the existing precondition
call (assertPendingStatus()) and replace direct parameter usages with command
accessors (e.g., this.name = command.name(), this.birthDate =
command.birthDate(), etc.), and update all call sites to construct and pass the
new command/builder; this preserves behavior while making the method concise and
easier to evolve.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: e043f6bf-560e-43e0-8605-480559d23430

📥 Commits

Reviewing files that changed from the base of the PR and between 1da641f and 5dad79e.

📒 Files selected for processing (2)

• src/main/java/org/sopt/kareer/domain/member/entity/Member.java
• src/main/java/org/sopt/kareer/domain/member/service/MemberService.java