Skip to content

[FIX] 일반 검색 시 특수문자 무시하도록 수정#248

Merged
rinarina0429 merged 3 commits intodevfrom
fix/#247
Jan 16, 2025
Merged

[FIX] 일반 검색 시 특수문자 무시하도록 수정#248
rinarina0429 merged 3 commits intodevfrom
fix/#247

Conversation

@rinarina0429
Copy link
Copy Markdown
Member

@rinarina0429 rinarina0429 commented Dec 31, 2024

Related Issue

Key Changes

  • Notion 검색 시 특수문자 무시 설정
  • 현재 검색 방식:
    "울어 봐"라고 검색하면 "울어봐"로 띄어쓰기를 없앤 뒤, 모든 띄어쓰기를 없앤 작품 제목들 중 "울어봐"를 포함한 작품들을 반환합니다. 그렇기 때문에 "울어봐 빌어도"라고 검색하면 "울어봐빌어도"를 포함한 것들을 검색하기 때문에 "울어 봐, 빌어도 좋고"라는 작품은 검색 결과로 나타나지 않습니다.
  • 변경된 검색 방식:
    그렇기 때문에 검색 시 특수문자를 무시하도록 검색 방식이 수정됐습니다.
    또한 기획 측에서 결정한 내용에 따라 특수문자만을 가지고 검색 시 빈 리스트를 반환하도록 수정하였습니다.

To Reviewers

  • 다음 스프린트, 혹은 다다음... 스프린트 중에 검색 방식을 아예 개편할 것 같습니다. 일단 임시방편으로 특수문자를 무시하도록 바꿨다는 점...

References

@rinarina0429 rinarina0429 self-assigned this Dec 31, 2024
@github-actions github-actions Bot requested review from ChaeAg and Kim-TaeUk December 31, 2024 13:03
@rinarina0429 rinarina0429 reopened this Jan 6, 2025
@rinarina0429 rinarina0429 marked this pull request as draft January 9, 2025 01:26
@rinarina0429 rinarina0429 marked this pull request as ready for review January 16, 2025 05:21
Copy link
Copy Markdown
Member

@ChaeAg ChaeAg left a comment

Choose a reason for hiding this comment

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

수고하셨습니당!
그런데 검색 조건에 어떤 개편이 있을 예정인건가여 ..? 😮

Comment thread src/main/java/org/websoso/WSSServer/service/NovelService.java
@Kim-TaeUk
Copy link
Copy Markdown
Member

수고하셨습니당! 그런데 검색 조건에 어떤 개편이 있을 예정인건가여 ..? 😮

저도 궁금쓰 ㅎㅎ

@rinarina0429 rinarina0429 merged commit 47b0802 into dev Jan 16, 2025
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.

[FIX] 작품 검색 시 특수문자 무시

3 participants