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

Feature/search by clothes api #44

Open
wants to merge 14 commits into
base: develop
Choose a base branch
from

Conversation

Dcaffein
Copy link
Collaborator

res가 너무 길어서 직접 테스트 해보심이 좋을 듯 합니다.
categories : Category[]
seasons : Season[]
name : string
Body에 들어갈 수 있는 key입니다.

src/controller/search/controller.ts Outdated Show resolved Hide resolved
src/controller/search/controller.ts Outdated Show resolved Hide resolved
Comment on lines 14 to 22
const categoriesArray = Array.isArray(categories)
? categories
: [categories].filter(Boolean);
const seasonsArray = Array.isArray(seasons)
? seasons
: [seasons].filter(Boolean);

//text를 String type으로
const textString = typeof text === 'string' ? text : '';
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분 제가 코멘트를 잘못 달아뒀던 것 같습니다. 죄송합니다ㅜㅜ 요청으로 배열이 필요한 경우에는 쿼리보다 바디를 이용하는 것이 더 좋을 것 같습니다..ㅜ

Copy link
Collaborator

@isseul isseul left a comment

Choose a reason for hiding this comment

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

reponse로 isWished도 필요할 것 같습니다..!

@isseul
Copy link
Collaborator

isseul commented Jan 15, 2024

지금 검색어에 따른 검색결과는 잘 나오는 것 같은데 category, season,filter 반영이 잘 안되는 것 같습니다ㅜㅜ

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.

3 participants