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: 인기검색어 조회 Client용 API 신규 생성 #380

Conversation

chris910512
Copy link
Contributor

@chris910512 chris910512 commented Dec 8, 2023

No description provided.

Copy link
Contributor

@JayPark7821 JayPark7821 left a comment

Choose a reason for hiding this comment

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

위 말씀드린 2가지 사항만 확인해 주시고
수정이 필요하다 판단하시면 수정해 주시고 머지해 주시면 될 것 같아요!!!

@@ -22,6 +22,9 @@ operation::v1/search/search[snippets='http-request,request-headers,request-param
=== 자동완성 검색어 추천 API [GET]
operation::v2/search/suggestions[snippets='http-request,request-parameters,http-response,response-fields']

=== 인기검색어 검색 테스트 API [GET]
Copy link
Contributor

Choose a reason for hiding this comment

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

테스트용 api 인건가용?!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

문서 내용 수정했습니당 (테스트 문자 제외)

@@ -32,7 +32,7 @@ public class HotKeyword extends BaseEntity {
private String keyword;
private int ranking;

private HotKeyword(final String keyword, final int ranking) {
public HotKeyword(final String keyword, final int ranking) {
Copy link
Contributor

Choose a reason for hiding this comment

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

해당 생성자를 호출하는 변경사항은 안보이는것같은데 public으로 변경하신 이유가 있을까용?!!
이왕이면 밑에 만들어놓은 스테틱 팩토리 메소드 사용해주시면 감사하겠습니닷!!!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

static method 활용하는걸로 변경했습니당

@chris910512 chris910512 merged commit c99e0b5 into develop Dec 11, 2023
1 check passed
@chris910512 chris910512 deleted the 345-feature-opensearch-도입-및-데이터-처리-프로그램-구현 branch December 11, 2023 23:53
@chris910512 chris910512 restored the 345-feature-opensearch-도입-및-데이터-처리-프로그램-구현 branch December 23, 2023 07:21
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.

2 participants