Skip to content

[DEPLOY] production-release(v.0.3.1)#366

Merged
taerimiiii merged 2 commits into
mainfrom
develop
Jun 17, 2026
Merged

[DEPLOY] production-release(v.0.3.1)#366
taerimiiii merged 2 commits into
mainfrom
develop

Conversation

@taerimiiii

@taerimiiii taerimiiii commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

🔗 Related Issue

✨ 작업 개요

작업 내용을 간략하게 작성해주세요.

v.0.3.1 배포!

체크리스트

  • Reviewers, Assignees, Labels를 모두 등록했나요?
  • .gitignore 설정을 하였나요?
  • PR 머지 전 반드시 CI가 정상적으로 작동하는지 확인해주세요!

📷 이미지 첨부 (선택)

  • 작업 결과를 확인할 수 있는 이미지나 GIF를 첨부해주세요.
  • UI 변경, API 응답 샘플, 테스트 결과 등이 포함되면 좋아요!

🧐 집중 리뷰 요청

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.

@taerimiiii taerimiiii requested a review from yaaan7 June 17, 2026 06:37
@taerimiiii taerimiiii self-assigned this Jun 17, 2026
@taerimiiii taerimiiii added the 🚀 deploy 배포 label Jun 17, 2026
@taerimiiii taerimiiii merged commit 1254bca into main Jun 17, 2026
2 checks passed

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds a new discount field to map pins, updating the repository queries, projection interfaces (MapPinView, MapPinClusterView), the response DTO (MapPinResDTO), and the mapping service. Feedback was provided to change the primitive double types for latitude and longitude in PinItemDTO to Double to prevent potential NullPointerException issues during auto-unboxing when values are null.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment on lines 10 to +14
double latitude, // 위도 (latitude)
double longitude, // 경도 (longitude)
String pinDetailAddress,
String pinLocation
String pinLocation,
String discount

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

현재 PinItemDTO에서 latitudelongitude가 기본 타입인 double로 정의되어 있습니다.

하지만 이 DTO로 변환되는 소스인 MapPinViewMapPinClusterView 인터페이스에서는 위경도 값을 래퍼 타입인 Double로 반환하고 있습니다. 만약 데이터베이스 조회 결과나 데이터 정합성 문제로 인해 위경도 값이 null로 반환되는 경우, 자바의 오토 언박싱(auto-unboxing) 과정에서 NullPointerException이 발생할 수 있습니다.

안정적인 예외 처리를 위해 double 대신 Double 타입을 사용하는 것을 권장합니다.

Suggested change
double latitude, // 위도 (latitude)
double longitude, // 경도 (longitude)
String pinDetailAddress,
String pinLocation
String pinLocation,
String discount
Double latitude, // 위도 (latitude)
Double longitude, // 경도 (longitude)
String pinDetailAddress,
String pinLocation,
String discount

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant