Skip to content

fix: 유저가 티켓팅 수령 정보 입력시 이름이 null로 변하는 문제 수정#279

Merged
doma17 merged 2 commits intomainfrom
develop
Oct 12, 2025
Merged

fix: 유저가 티켓팅 수령 정보 입력시 이름이 null로 변하는 문제 수정#279
doma17 merged 2 commits intomainfrom
develop

Conversation

@doma17
Copy link
Copy Markdown
Member

@doma17 doma17 commented Oct 12, 2025

#️⃣ 연관된 이슈

ex) #이슈번호, #이슈번호

#278

Summary by CodeRabbit

  • Refactor
    • 참여 정보 업데이트에서 ‘이름’ 항목을 제거하여 이제 학번과 학과만 수정됩니다.
    • 관련 화면/요청에서 이름 입력란이 더 이상 표시되지 않으며, 해당 필드에 대한 검증/안내 문구가 정리되었습니다.
    • 기존에 저장된 프로필의 이름 값은 변경되지 않고 그대로 유지됩니다.

@doma17 doma17 self-assigned this Oct 12, 2025
@doma17 doma17 added the 🐞 BugFix 버그 발생 또는 해결 label Oct 12, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 12, 2025

📝 Walkthrough

Walkthrough

사용자 티켓팅 참여 정보 업데이트 요청 DTO에서 name 필드를 제거했고, UserEntity의 updateParticipationInfo 메서드에서 name 업데이트 로직을 삭제하여 studentId와 department만 갱신하도록 변경되었습니다.

Changes

Cohort / File(s) Summary of Changes
DTO 필드 제거
src/main/java/inu/codin/codin/domain/user/dto/request/UserTicketingParticipationInfoUpdateRequest.java
요청 DTO에서 name 필드 제거로 직렬화/역직렬화 스키마 축소
엔티티 업데이트 로직 정리
src/main/java/inu/codin/codin/domain/user/entity/UserEntity.java
updateParticipationInfo에서 name 갱신 코드 삭제; studentId, department만 업데이트

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

깡총깡총, 이름은 내려놓고
학번과 학과만 주머니에 쏙!
필드 하나 줄였더니 바람이 산뜻—
코드밭 고요해지고 빌드도 굴벵굴벵.
오늘도 토끼는 PR 위에 앉아, “깨끗!” 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning 현재 설명에는 연관된 이슈만 언급되어 있고 템플릿에 요구된 “📝 작업 내용” 섹션과 리뷰 요구사항 등이 빠져 있어 PR의 구체적 변경 사항과 검토 포인트를 파악하기 어렵습니다. 템플릿에 따라 작업한 내용을 요약한 “📝 작업 내용” 섹션을 추가하고 필요시 스크린샷이나 리뷰어에게 원하는 검토 포인트를 포함해 설명을 완성해주세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed 제목이 이번 변경사항의 핵심인 “유저 이름이 null로 변하는 문제 수정”을 명확히 요약하고 있어 변경 내용과 완전히 일치하며 간결하고 구체적입니다.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 645422f and f9d2000.

📒 Files selected for processing (2)
  • src/main/java/inu/codin/codin/domain/user/dto/request/UserTicketingParticipationInfoUpdateRequest.java (0 hunks)
  • src/main/java/inu/codin/codin/domain/user/entity/UserEntity.java (0 hunks)
💤 Files with no reviewable changes (2)
  • src/main/java/inu/codin/codin/domain/user/dto/request/UserTicketingParticipationInfoUpdateRequest.java
  • src/main/java/inu/codin/codin/domain/user/entity/UserEntity.java

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

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

@doma17 doma17 merged commit 13b0d3c into main Oct 12, 2025
1 check passed
@coderabbitai coderabbitai Bot mentioned this pull request Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 BugFix 버그 발생 또는 해결

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant