Skip to content

refactor: 학과 정보 및 주점 정보 추가 #358

Merged
Jjiggu merged 2 commits intodevelopfrom
feature/#348-Waiting-Rebuild
Feb 1, 2026
Merged

refactor: 학과 정보 및 주점 정보 추가 #358
Jjiggu merged 2 commits intodevelopfrom
feature/#348-Waiting-Rebuild

Conversation

@Jjiggu
Copy link
Contributor

@Jjiggu Jjiggu commented Feb 1, 2026

작업 요약

학과 정보 및 주점 정보 필드 추가

Issue Link

문제점 및 어려움

해결 방안

Reference

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 대기 인원 조회 응답에 매장명 및 학과명 정보 추가
  • 버그 수정

    • 존재하지 않는 학과 조회 시 명확한 오류 메시지 제공

✏️ Tip: You can customize this high-level summary in your review settings.

@Jjiggu Jjiggu self-assigned this Feb 1, 2026
@Jjiggu Jjiggu added the refactor 리팩토링 label Feb 1, 2026
@Jjiggu Jjiggu merged commit c1acf92 into develop Feb 1, 2026
1 check passed
@github-actions github-actions bot requested a review from HyemIin February 1, 2026 14:12
@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

Caution

Review failed

The pull request is closed.

워크스루

애플리케이션 사용자 API에 Department 도메인 기능을 통합합니다. 새로운 DepartmentNotFoundException을 정의 및 전역 처리하며, WaitingService를 강화하여 부서 정보를 조회하고 응답에 포함합니다.

변경사항

코호트 / 파일 요약
예외 처리
nowait-domain/.../DepartmentNotFoundException.java, nowait-common/.../ErrorMessage.java, nowait-app-user-api/.../GlobalExceptionHandler.java
새로운 DepartmentNotFoundException 예외 클래스 생성, ErrorMessage에 DEPARTMENT_NOT_FOUND 상수 추가, GlobalExceptionHandler에서 해당 예외 처리 메서드 추가
DTO 확장
nowait-app-user-api/.../GetWaitingSizeResponse.java
storeName 및 departmentName 필드 추가하여 응답 페이로드 확장
서비스 로직
nowait-app-user-api/.../WaitingService.java
DepartmentRepository 의존성 주입, 부서 정보 조회 로직 추가, GetWaitingSizeResponse에 부서명 및 매장명 포함

시퀀스 다이어그램

sequenceDiagram
    participant Client
    participant WaitingService
    participant StoreRepository
    participant DepartmentRepository
    participant Database

    Client->>WaitingService: getWaitingCount(storeId)
    WaitingService->>StoreRepository: findById(storeId)
    StoreRepository->>Database: SELECT store WHERE id = ?
    Database-->>StoreRepository: Store
    StoreRepository-->>WaitingService: Store
    
    WaitingService->>DepartmentRepository: findById(store.departmentId)
    DepartmentRepository->>Database: SELECT department WHERE id = ?
    Database-->>DepartmentRepository: Department
    DepartmentRepository-->>WaitingService: Department
    
    WaitingService->>WaitingService: Build GetWaitingSizeResponse<br/>(storeId, waitingCount,<br/>storeName, departmentName)
    WaitingService-->>Client: GetWaitingSizeResponse
Loading

예상 코드 리뷰 난이도

🎯 3 (중간) | ⏱️ ~20분

관련 가능성 있는 PR

제안 리뷰어

  • HyemIin
✨ 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 feature/#348-Waiting-Rebuild

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.

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