Conversation
[FEAT]: 식당(Store) 도메인 엔티티 설계
[FEAT]: HTTPS 적용 및 NGINX Blue-Green 무중단 배포 전환 로직 수정
main 브랜치와 코드 통일
[FIX]: 불필요 도커이미지 제거 명령어 배포 스크립트에 추가
Readme 회원정보 수정
* FEAT: 사업자 인증 시 대표자 성명 입력 필드 추가 및 검증 반영 * FEAT: 가게 등록 시 사업자번호 대표 성명 입력 필드 추가 및 검증 로직 반영
📝 WalkthroughWalkthrough스토어 검색 DTO에 위도(latitude)와 경도(longitude) 필드를 추가하고, 변환기(converter)를 업데이트하여 Store 엔티티에서 이 필드들을 채우도록 변경되었습니다. 사용하지 않는 import도 제거되었습니다. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (1 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/main/java/com/eatsfine/eatsfine/domain/store/dto/StoreResDto.java`:
- Around line 31-35: StoreDetailDto is missing latitude/longitude while
StoreSearchDto gained them; add double latitude and double longitude fields to
the StoreDetailDto class (alongside existing fields like mainImageUrl and
isOpenNow), include them in the StoreDetailDto constructor (or builder/factory
such as any existing of()/fromEntity() method), and ensure the mapping code that
builds StoreDetailDto from the Store entity populates these fields; if omission
was intentional, add a clear code comment in StoreDetailDto explaining why
coordinates are omitted.
| double latitude, | ||
| double longitude, | ||
| String mainImageUrl, | ||
| boolean isOpenNow | ||
| ){} |
There was a problem hiding this comment.
🧹 Nitpick | 🔵 Trivial
StoreDetailDto에 좌표 필드 누락 — API 일관성 검토 필요
StoreSearchDto(검색 목록)에는 latitude/longitude가 추가되었지만, StoreDetailDto(상세 조회, line 54-75)에는 없습니다. 클라이언트가 상세 페이지에서 지도 마커를 표시할 때 별도 검색 호출이 필요해지는 문제가 생깁니다. 의도적인 설계라면 코드 주석으로 이유를 명시해두는 것을 권장합니다.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/main/java/com/eatsfine/eatsfine/domain/store/dto/StoreResDto.java` around
lines 31 - 35, StoreDetailDto is missing latitude/longitude while StoreSearchDto
gained them; add double latitude and double longitude fields to the
StoreDetailDto class (alongside existing fields like mainImageUrl and
isOpenNow), include them in the StoreDetailDto constructor (or builder/factory
such as any existing of()/fromEntity() method), and ensure the mapping code that
builds StoreDetailDto from the Store entity populates these fields; if omission
was intentional, add a clear code comment in StoreDetailDto explaining why
coordinates are omitted.
💡 작업 개요
✅ 작업 내용
🧪 테스트 내용
📝 기타 참고 사항
Summary by CodeRabbit
새로운 기능