Skip to content

feat: 주변 상점 검색 키워드 API 추가 & 주변 상점 정보 조회 API V2 추가 #2051

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Nov 2, 2025
Merged

feat: 주변 상점 검색 키워드 API 추가 & 주변 상점 정보 조회 API V2 추가 #2051

merged 4 commits into from
Nov 2, 2025

Conversation

@DHkimgit
Copy link
Collaborator

@DHkimgit DHkimgit commented Nov 1, 2025

🔍 개요

  • 주변 상점 검색 키워드 API 추가
  • 주변 상점 정보 조회 API V2 추가

🚀 주요 변경 내용

💬 참고 사항

✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

@DHkimgit DHkimgit self-assigned this Nov 1, 2025
@DHkimgit DHkimgit added 기능 새로운 기능을 개발합니다. Team Business 비즈니스 팀에서 작업할 이슈입니다 labels Nov 1, 2025
@github-actions
Copy link

github-actions bot commented Nov 1, 2025 •
edited
Loading

Unit Test Results

668 tests   667 ✔️  1m 20s ⏱️
164 suites      1 💤
164 files        0 ❌

Results for commit 7a5ac9e.

♻️ This comment has been updated with latest results.

asa9874
asa9874 reviewed Nov 1, 2025
View reviewed changes
Copy link
Member

@asa9874 asa9874 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.😀

Soundbar91
Soundbar91 requested changes Nov 1, 2025
View reviewed changes
Copy link
Collaborator

@Soundbar91 Soundbar91 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !

주변 상점 정보 조회 API에 응답값에 openTime, closeTime 필드 추가하는 거라면 기존 API에 추가해도 괜찮을 거 같은데 버저닝을 진행하신 이유가 궁금합니다 !

@DHkimgit
Copy link
Collaborator Author

DHkimgit commented Nov 1, 2025

고생하셨습니다 !

주변 상점 정보 조회 API에 응답값에 openTime, closeTime 필드 추가하는 거라면 기존 API에 추가해도 괜찮을 거 같은데 버저닝을 진행하신 이유가 궁금합니다 !

이게 프론트, IOS, 안드로이드 전부 사용하는 API고 버전이 다양해서 혹여나 클라이언트 라이브러리중에 오류가 날 수 있는 버전이 있을 수도 있다고 생각해서 버저닝으로 처리했습니다.

Soundbar91
Soundbar91 approved these changes Nov 1, 2025
View reviewed changes
Copy link
Collaborator

@Soundbar91 Soundbar91 left a comment

Choose a reason for hiding this comment

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

고생하셨습니답 !

...java/in/koreatech/koin/domain/shop/dto/search/response/ShopSearchRelatedKeywordResponse.java Outdated
Comment on lines 44 to 45
orderableShopNameKeywordHit.shopId(),
orderableShopNameKeywordHit.shopName()
Copy link
Collaborator

@Soundbar91 Soundbar91 Nov 1, 2025

Choose a reason for hiding this comment

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

C

요기 변수 네이밍이 orderableShop 입니당

...java/in/koreatech/koin/domain/shop/dto/search/response/ShopSearchRelatedKeywordResponse.java Outdated
Comment on lines 66 to 68
orderableShopMenuNameKeywordHit.shopId(),
orderableShopMenuNameKeywordHit.shopName(),
orderableShopMenuNameKeywordHit.menuName()
Copy link
Collaborator

@Soundbar91 Soundbar91 Nov 1, 2025

Choose a reason for hiding this comment

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

C

요기 변수 네이밍이 orderableShop 입니당 22

asa9874
asa9874 approved these changes Nov 1, 2025
View reviewed changes
Copy link
Member

@asa9874 asa9874 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 👍

한가지 궁금한 점이 현재 SnakeCaseStrategy클래스를 프로젝트 내에서 import static와 import가 혼용되어 사용되고 있는거같은데, SnakeCaseStrategy 클래스를 import static 해서 사용하는 이유가 혹시 있나요?

@DHkimgit
Copy link
Collaborator Author

DHkimgit commented Nov 2, 2025

고생하셨습니다. 👍

한가지 궁금한 점이 현재 SnakeCaseStrategy클래스를 프로젝트 내에서 import static와 import가 혼용되어 사용되고 있는거같은데, SnakeCaseStrategy 클래스를 import static 해서 사용하는 이유가 혹시 있나요?

이거 static import 하는게 규칙이라고 알고있습니다.

@DHkimgit DHkimgit merged commit 3b88e57 into develop Nov 2, 2025
5 checks passed
@DHkimgit DHkimgit deleted the feat/2050-shop-search-related-keyword branch November 2, 2025 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Soundbar91 Soundbar91 Soundbar91 approved these changes

@asa9874 asa9874 asa9874 approved these changes

@ImTotem ImTotem Awaiting requested review from ImTotem

Assignees

@DHkimgit DHkimgit

Labels

Team Business 비즈니스 팀에서 작업할 이슈입니다 기능 새로운 기능을 개발합니다.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[비즈니스] 주변 상점 키워드 검색 API 추가 및 응답값 수정

4 participants

@DHkimgit @Soundbar91 @asa9874