Add: support maxPartLength in /rooms/search #100
Merged
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
Closed
1 task
withSang
commented
Aug 11, 2022
withSang
commented
Aug 11, 2022
GrasshopperBears
approved these changes
Aug 12, 2022
|
|
||
| // 검색 쿼리를 설정합니다. | ||
| const query = {}; | ||
| if (name) query.name = { $regex: new RegExp(name, "i") }; // 'i': 대소문자 무시 |
There was a problem hiding this comment.
OTL에서는 띄어쓰기가 검색결과에 영향을 미치지 않게 변경한 적이 있습니다.(예: 방 이름이 초보만 들어와인 경우 초보만들어와로 검색이 안됨)
방 이름도 그런 방식으로 해보는 걸 고려해도 좋을 것 같아요~
There was a problem hiding this comment.
오 추가되면 검색할 때 편하겠어요.
mongoose에서 Virtual field를 기준으로 쿼리하는 걸 지원하지는 않더라고요. 그래도 mongoDB 데이터베이스 레벨에서 정규 표현식 쿼리를 지원하니까 공백을 기준으로 구분한 각각의 단어를 모두 포함하는 제목의 방들만 반환하도록 정규 표현식을 짜면 간단하겠어요! 다음 이슈에서 해보겠습니다
14KGun
reviewed
Aug 14, 2022
Comment on lines
+519
to
+522
| removeAllRoomHandler, | ||
| idSettlementHandler, | ||
| idEditHandler, | ||
| idDeleteHandler, |
There was a problem hiding this comment.
removeAllRoomHandler, idEditHandler, idDeleteHandler 와 같은 핸들러들이 test에서만 작동할 수 호출될 수 있도록 하는 미들웨어나 로직이 있나요???
There was a problem hiding this comment.
오 저도 그게 지금 없어서 추가하고 싶어요!
저 API들이 admin 페이지와 역할이 겹쳐서 아예 지우는 것도 괜찮다고 생각해요
Open
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.
Summary
It closes #99
방 검색 API(/rooms/search)에서 방의 최대 인원 수를 요청에 추가하는 것을 지원합니다.
최대 인원 수(2~4명)이 요청에 들어왔을 때, 해당 최대 인원 수와 방의 최대 인원 수 속성이 일치할 때에만 해당 방을 반환합니다.
최대 인원 수가 주어지지 않은 경우, 임의의 최대 인원 수를 가지는 방들을 반환합니다.
Extra info
Images
Further Work