Skip to content

Conversation

@oroi2009
Copy link
Contributor

@oroi2009 oroi2009 commented Aug 14, 2025

🔍 관련 이슈


✅ 작업 분류

  • 버그 수정
  • 신규 기능
  • 프로젝트 구조 변경
  • 코드 리팩토링
  • 기능 수정

✨ 작업 내용

  • 근처 가게 위치 목록 조회 API 구현

👥 전달사항

  1. 반찬 매물이 없는 가게는 목록에 포함되지 않습니다.
  2. 테스트 용이성, 유지보수성을 고려하여 쿼리문에서 NOW() 사용이 아닌, 파라미터를 넘기는 방식으로 변경하였습니다.

✅ 체크리스트

  • 코드가 컴파일 및 빌드됨
  • 모든 테스트가 통과함
  • 관련 문서가 업데이트됨
  • 코드 리뷰가 수행됨
  • 커밋 메시지를 확인함

📸 스크린샷

image

@oroi2009 oroi2009 requested a review from frombunny August 14, 2025 07:26
@oroi2009 oroi2009 self-assigned this Aug 14, 2025
Copy link
Member

@frombunny frombunny left a comment

Choose a reason for hiding this comment

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

저것만 바꾸고 바로 병합하시면 될 것 같아요!

List<Store> findNearbyStores(
double lat,
double lng,
LocalDateTime now,
Copy link
Member

Choose a reason for hiding this comment

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

이건 굳이 안 넘겨줘도 될 듯! NOW() 사용 중이라!

Comment on lines 22 to 28
double lat,
double lng,
LocalDateTime now,
double minLat,
double maxLat,
double minLng,
double maxLng
Copy link
Member

Choose a reason for hiding this comment

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

Java 컴파일 시 기본적으로 파라미터 이름이 유지되지 않을 수 있어서 파라미터에 아래처럼 어노테이션 붙여주는 게 좋대요!
@param("lat") double lat,
@param("lng") double lng,
... 이런 식으로!

@oroi2009 oroi2009 merged commit 3a375f1 into main Aug 15, 2025
2 checks passed
@frombunny frombunny deleted the feat/#19 branch August 25, 2025 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

✨ feat : 근처 가게 위치 목록 조회 API 개발

3 participants