Skip to content
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.

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

[feat] 식당 검색을 위한 네이버 검색 api 연결 #57

Merged
merged 6 commits into from
Jul 13, 2024

Conversation

Parkjyun
Copy link
Contributor

Related Issue 📌

close #56

Description ✔️

  • feign client를 사용하여 네이버 api를 연결하였습니다.
  • 좌표 스펙을 맞추기 위해 간단한 연산을 추가하고 api에 맞게 응답 구조를 변경했습니다.

To Reviewers

@Parkjyun Parkjyun self-assigned this Jul 12, 2024
NaverLocationsDto locationsDto = naverFeignClient.getLocationInfo(clientId, clientSecret, query, DEFAULT_SEARCH_SIZE);
return LocationsResponse.of(locationsDto.items().stream()
.map(LocationResponse::of)
.collect(Collectors.toList()));
Copy link
Contributor

Choose a reason for hiding this comment

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

.collect(Collectors.toList())); 대신 toList()로 써도 될 것 같습니당

Copy link
Member

@kgy1008 kgy1008 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다


private final NaverFeignClient naverFeignClient;

public LocationsResponse getLocations(String query) {
Copy link
Member

Choose a reason for hiding this comment

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

final 키워드 붙여주세요


@Service
@RequiredArgsConstructor
public class ExternalService {
Copy link
Member

Choose a reason for hiding this comment

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

저희 카카오나 애플 로그인도 외부 서비스여서 ExternalService보다는 네이버 검색 api를 연결하는 서비스라는 것을 들어낼 수 있도록 클래스명을 변경하는 것이 좋아보여용

@Parkjyun Parkjyun merged commit 4ac18e4 into develop Jul 13, 2024
1 check passed
@kgy1008 kgy1008 deleted the feat/56 branch December 2, 2024 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 장소 검색 api 생성
3 participants