Skip to content

Conversation

@eraser502
Copy link
Contributor

Overview

  • 실서버 배포

Related Issue

  • Closes

PR Checklist

  • [ ]

Additional Information

@coderabbitai
Copy link

coderabbitai bot commented Dec 12, 2025

Caution

Review failed

The pull request is closed.

개요

이 변경사항은 상담(consultation) 도메인을 확장하여 날짜 범위 검색 필터링, 새로운 CANCELLED 상태, 그리고 remark 필드를 추가합니다. ConsultationSearchConditionstartDateendDate를 도입하고, 상담 엔티티와 업데이트 요청 DTO에 비고(remark) 필드를 추가하며, 레포지토리 쿼리 로직을 개선하여 날짜 범위 필터링과 null 상태 처리(CANCELLED 제외)를 구현합니다. DB 마이그레이션으로 테이블 스키마를 업데이트하고, 컨트롤러와 서비스 레이어도 이에 맞춰 수정됩니다.

코드 리뷰 예상 난이도

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

  • 주의가 필요한 영역:
    • ConsultationRepositoryImpl의 새로운 상태 조건 처리 로직: null 상태일 때 CANCELLED을 제외하는 동작 검증 필요
    • createdAtGoe()createdAtLt() 날짜 범위 필터링 로직의 정확성 확인
    • DB 마이그레이션의 idempotence 및 기존 데이터와의 호환성 검증
    • ConsultationUpdateRequest에 추가된 remark 필드의 검증 규칙(1000자 제한) 확인

관련 가능 PR

제안 라벨

feature

🐰 날짜 필터로 상담 검색해,
비고(remark) 필드 담아 정보 풍성하게,
CANCELLED 상태 추가되니,
쿼리 로직 개선되고 흐름 매끄러워,
데이터 구조 한층 완벽해졌네! ✨

✨ 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 425c5a9 and f38587e.

📒 Files selected for processing (10)
  • src/main/java/redot/redot_server/domain/admin/controller/AdminConsultationController.java (1 hunks)
  • src/main/java/redot/redot_server/domain/admin/controller/docs/AdminConsultationControllerDocs.java (1 hunks)
  • src/main/java/redot/redot_server/domain/admin/dto/ConsultationSearchCondition.java (2 hunks)
  • src/main/java/redot/redot_server/domain/admin/dto/request/ConsultationUpdateRequest.java (1 hunks)
  • src/main/java/redot/redot_server/domain/admin/service/AdminConsultationService.java (3 hunks)
  • src/main/java/redot/redot_server/domain/redot/consultation/dto/response/ConsultationResponse.java (2 hunks)
  • src/main/java/redot/redot_server/domain/redot/consultation/entity/Consultation.java (2 hunks)
  • src/main/java/redot/redot_server/domain/redot/consultation/entity/ConsultationStatus.java (1 hunks)
  • src/main/java/redot/redot_server/domain/redot/consultation/repository/ConsultationRepositoryImpl.java (3 hunks)
  • src/main/resources/db/migration/V4__add_consultation_remark_and_cancelled_status.sql (1 hunks)

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.

@eraser502 eraser502 merged commit df9838b into main Dec 12, 2025
1 of 2 checks passed
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.

2 participants