상품검색 페이지 #105
Merged
상품검색 페이지 #105
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 키워드 검색을 위한 컴포넌트 추가
- 오탈자 보정이나, 연관검색어 제공을 위해서 별도의 검색페이지를 만들어서 검색하는 페이지 추가(라우터)
- 한글을 입력해야하는데 영타로 작성한 경우 검색어 제안을 위해서 inko 라이브러리 설치 - 한글자인 경우에는 아직 입력중이라 생각되기 때문에 변환된 글자가 두 글자를 넘어서는 경우에만 추천할 수 있도록 변경
- 사용자가 입력한 검색어를 실제 검색결과에 반영하기 위해서 context - 검색결과 초기화는 별도의 버튼을 이용하기 때문에 여기서는 처리하지 않는다.
kgpyo
added
feature
Johnie-Yeo
requested changes
Dec 18, 2019
web-apps/client/src/pages/search.js
Outdated
| } | ||
| if (isKoreanOnly(keyword)) { | ||
| recommendKeyword = new Inko().ko2en(keyword); | ||
| } |
There was a problem hiding this comment.
이 부분에 대해서는 추가적으로 개선작업이 반드시 필요할 것으로 생각됩니다.
한글과 영어가 섞이는 과정은 꽤 많을 것으로 예상됩니다.
ex) 갤럭시S10 -> roffjrtlㄴ7
There was a problem hiding this comment.
혼동을 피하는 메소드로 변경하겠습니다.
해당 메소드는 영어만 사용되지 않으면 변환됩니다.
영어 섞이는 경우는 다른 방법으로 고려해보겠습니다.
web-apps/client/src/pages/search.js
Outdated
| params: { keyword } | ||
| }) | ||
| return response.data; | ||
| } |
There was a problem hiding this comment.
api 호출은 custom Hooks 등의 방법으로 외부로 분리하여야 할 것 같네요. uri는 또 별도의 파일로 관리가 필요하겠고요.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR에 대한 요약
PR에 대한 동기와 상황에 대한 설명 부탁드립니다.
-일래스틱 서치 검색(#55)을 이용하여 사용자가 키워드를 입력할 때 한영타가 바뀌거나 한두글자 정도의 오탈자를 보정하여 검색할 수 있게 제공합니다. #45
이번 코드는 어떻게 테스트 되었나요?
결과물 스크린샷 (선택) :
어떤 변화인가요?
체크리스트:
본 PR은 다음 이슈에 해당하는 내용입니다.
Closes #55 #46 #45