Skip to content

refactor: reservationId 필드명 waitingNumber로 일관화#364

Merged
Jjiggu merged 1 commit intodevelopfrom
refactor#361-getMyWaiting
Feb 13, 2026
Merged

refactor: reservationId 필드명 waitingNumber로 일관화#364
Jjiggu merged 1 commit intodevelopfrom
refactor#361-getMyWaiting

Conversation

@Jjiggu
Copy link
Contributor

@Jjiggu Jjiggu commented Feb 13, 2026

작업 요약

reservationId 필드명 waitingNumber로 일관화

Issue Link

#361

문제점 및 어려움

해결 방안

Reference

Summary by CodeRabbit

릴리스 노트

  • Refactor
    • 대기 정보 조회 API 응답 구조가 변경되었습니다. API를 사용하는 클라이언트에서 응답 필드를 업데이트해야 합니다.

@Jjiggu Jjiggu self-assigned this Feb 13, 2026
@Jjiggu Jjiggu added the refactor 리팩토링 label Feb 13, 2026
@github-actions github-actions bot requested a review from HyemIin February 13, 2026 11:09
@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

개요

예약 ID를 기반으로 한 필드명을 대기 번호로 변경하는 리팩토링입니다. GetMyWaitingInfoResponse, WaitingService, GetMyWaitingBaseDto의 필드명과 생성자 파라미터가 reservationId에서 waitingNumber로 통일됩니다.

변경 사항

코호트 / 파일 요약
DTO 필드 리네이밍
com/nowait/applicationuser/waiting/dto/GetMyWaitingInfoResponse.java, com/nowait/domaincorerdb/reservation/dto/GetMyWaitingBaseDto.java
reservationId 필드를 waitingNumber로 변경. GetMyWaitingBaseDto는 @QueryProjection 생성자 파라미터도 함께 업데이트.
서비스 레이어 매핑 업데이트
com/nowait/applicationuser/waiting/service/WaitingService.java
getMyWaitingInfo 메서드에서 응답 빌더의 필드 매핑을 reservationId()에서 waitingNumber()로 변경.

예상 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~10 분

제안 검토자

  • HyemIin
🚥 Pre-merge checks | ✅ 2 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Merge Conflict Detection ⚠️ Warning ⚠️ Unable to check for merge conflicts: Invalid branch name format
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 변경사항의 핵심을 명확하게 요약하고 있습니다. reservationId 필드명을 waitingNumber로 일관화한다는 주요 변경사항을 정확히 반영하고 있습니다.

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

✨ 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 refactor#361-getMyWaiting
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch refactor#361-getMyWaiting
  • Create stacked PR with resolved conflicts
  • Post resolved changes as copyable diffs in a comment

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
nowait-app-user-api/src/main/java/com/nowait/applicationuser/waiting/dto/GetMyWaitingInfoResponse.java (1)

11-11: 사소한 포맷 이슈: StringwaitingNumber 사이에 공백이 두 개 있습니다.

-	private String  waitingNumber;
+	private String waitingNumber;

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.

@Jjiggu Jjiggu merged commit dba48bc into develop Feb 13, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant