feat: 개인 일정에 색상(color) 필드 추가 - #334
Conversation
- MemberSchedule 엔티티에 color 컬럼 추가 (HEX 형식, 최대 7자) - ScheduleDetails 응답 DTO에 color 필드 포함 - 일정 생성/수정 요청 DTO에 color 필드 및 HEX 형식 유효성 검사 추가 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Warning Review limit reached
More reviews will be available in 45 minutes and 37 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
📝 Walkthrough요약일정 엔티티에 HEX 형식 검증이 포함된 색상(color) 필드가 요청부터 응답까지 전체 스택에 걸쳐 추가되었습니다. 변경 사항Member Schedule Color Field Support
예상 코드 리뷰 소요 시간🎯 2 (Simple) | ⏱️ ~10 분 시 (Poem)
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
src/main/java/com/dongsoop/dongsoop/calendar/entity/MemberSchedule.java (1)
53-54:member_schedule.color컬럼 반영 방식(마이그레이션 여부) 확인리포지토리에서 Flyway/Liquibase 의존성 및
V*__*.sql,changelog*.xml같은 마이그레이션 리소스는 확인되지 않았고, 운영 프로필(application-prod.yml)에서도spring.jpa.hibernate.ddl-auto: update로 스키마를 자동 변경하도록 설정돼 있습니다. 따라서 마이그레이션 누락으로 인한 즉시 SQL 오류 가능성은 낮지만, 운영 배포 시에도 동일 프로파일/설정 적용 여부와 DB 계정 권한으로 ALTER가 가능한지 확인이 필요합니다.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/main/java/com/dongsoop/dongsoop/calendar/entity/MemberSchedule.java` around lines 53 - 54, 확인: 엔티티 필드 MemberSchedule.color가 데이터베이스에 반영되는 방법을 명확히 하세요; 현재 리포지토리에 Flyway/Liquibase 마이그레이션 파일(V*__*.sql 또는 changelog*.xml)이 없고 운영 설정(application-prod.yml)에 spring.jpa.hibernate.ddl-auto: update가 사용되므로 배포 환경에서도 같은 프로파일이 적용되는지, 운영 DB 계정이 ALTER 권한을 가지고 있는지 확인하고 문서화하세요; 권한 또는 프로파일 불확실시에는 MemberSchedule.color 컬럼 추가용 마이그레이션 스크립트(또는 changelog)를 생성하여 배포 파이프라인에 포함시키고, 배포 전 스테이징에서 적용 테스트를 수행하도록 변경하세요.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@src/main/java/com/dongsoop/dongsoop/calendar/entity/MemberSchedule.java`:
- Around line 53-54: 확인: 엔티티 필드 MemberSchedule.color가 데이터베이스에 반영되는 방법을 명확히 하세요;
현재 리포지토리에 Flyway/Liquibase 마이그레이션 파일(V*__*.sql 또는 changelog*.xml)이 없고 운영
설정(application-prod.yml)에 spring.jpa.hibernate.ddl-auto: update가 사용되므로 배포 환경에서도
같은 프로파일이 적용되는지, 운영 DB 계정이 ALTER 권한을 가지고 있는지 확인하고 문서화하세요; 권한 또는 프로파일 불확실시에는
MemberSchedule.color 컬럼 추가용 마이그레이션 스크립트(또는 changelog)를 생성하여 배포 파이프라인에 포함시키고, 배포
전 스테이징에서 적용 테스트를 수행하도록 변경하세요.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 87efc793-b2c3-4ceb-a290-f7036f1f5132
📒 Files selected for processing (4)
src/main/java/com/dongsoop/dongsoop/calendar/dto/CreateMemberScheduleRequest.javasrc/main/java/com/dongsoop/dongsoop/calendar/dto/MemberScheduleUpdateRequest.javasrc/main/java/com/dongsoop/dongsoop/calendar/dto/ScheduleDetails.javasrc/main/java/com/dongsoop/dongsoop/calendar/entity/MemberSchedule.java
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 participant
Summary
MemberSchedule엔티티에color컬럼 추가 (HEX 형식,#RRGGBB또는#RGB, 최대 7자, nullable)ScheduleDetails응답 DTO에color필드 포함 → 리스트/상세 조회 시 색상 반환CreateMemberScheduleRequest에color필드 추가 및 HEX 형식 유효성 검사 (@Pattern)MemberScheduleUpdateRequest에color필드 추가 및 HEX 형식 유효성 검사 (@Pattern)MemberSchedule.toEntity(),toDetails(),update()메서드에 color 반영Test plan
POST /schedule/member—color: "#FF5733"포함 요청 시 정상 생성 확인POST /schedule/member—color: "invalid"포함 요청 시 400 응답 확인PATCH /schedule/member/{id}— color 수정 후 조회 시 변경 반영 확인GET /schedule/{yearMonth}— 응답 DTO에color필드 포함 확인🤖 Generated with Claude Code
Summary by CodeRabbit
릴리스 노트
#RRGGBB또는#RGB)의 색상을 지원하며, 일정 생성 및 수정 시 색상을 설정하고 변경할 수 있습니다.