-
Notifications
You must be signed in to change notification settings - Fork 1
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
The head ref may contain hidden characters: "345-feature-opensearch-\uB3C4\uC785-\uBC0F-\uB370\uC774\uD130-\uCC98\uB9AC-\uD504\uB85C\uADF8\uB7A8-\uAD6C\uD604"
feat: 인기검색어 조회 Client용 API 신규 생성 #380
Conversation
…github.com/daedongbread/bread-map-backend-2 into 345-feature-opensearch-도입-및-데이터-처리-프로그램-구현
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
위 말씀드린 2가지 사항만 확인해 주시고
수정이 필요하다 판단하시면 수정해 주시고 머지해 주시면 될 것 같아요!!!
src/docs/asciidoc/search.adoc
Outdated
@@ -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] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
테스트용 api 인건가용?!!
There was a problem hiding this comment.
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) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
해당 생성자를 호출하는 변경사항은 안보이는것같은데 public으로 변경하신 이유가 있을까용?!!
이왕이면 밑에 만들어놓은 스테틱 팩토리 메소드 사용해주시면 감사하겠습니닷!!!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
static method 활용하는걸로 변경했습니당
No description provided.