-
Notifications
You must be signed in to change notification settings - Fork 0
[OD-179] 회원가입 시 유저 태그 설정 #132
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
Conversation
lalaurrel
left a comment
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.
고생하셨습니다 미미재양 🫧
| style: { mobile: 'title3-bold', tablet: 'title2-bold', desktop: 'title1-bold' }, | ||
| }} | ||
| > | ||
| 스타일을 선택해주세요! |
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.
${nickname}님의 OOTD를 선택해주세요! 는 어떠한지!!!
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.
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.
후자로 갑시다
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.
수정했습니다~!
| style: { mobile: 'caption1-medium', tablet: 'body2-medium', desktop: 'body1-medium' }, | ||
| }} | ||
| > | ||
| OODD가 당신의 취향을 분석하여 맞춤 스타일을 추천해 드릴게요. |
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.
자신이 고른 옷 스타일과 유사한 이성을 추천해주는건가요?!
이것도 자신의 스타일 태그를 선택하는 페이지랑 자신이 원하는 이성의 스타일을 선택하는 페이지를 세분화해두면 나중에 홈에 띄울 때나 추천해줄 때 더 수월하고
유저 입장에서도 자기가 원하는 스타일을 직접 선택하고 추천받을 수 있어서 좋을 것 같다는 생각!
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.
네넹 스트릿, 페미닌 등을 골랐으면 해당 스타일의 게시글을 가진 이성을 추천해주는 거라 이해했습니당
근데 자신의 스타일 태그를 회원가입 때 선택하는 건 좋은 의견인 것 같아요!!! 현재는 그냥 게시글의 태그를 바탕으로 남의 스타일을 분석해서 추천해 주는 건데 게시글의 스타일이 너무 중구난방일 수도 있으니 😅 @happbob
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.
좋은 의견입니다. 다만 스타일 태그를 회원가입 때 선택할 때 그 스타일이 어떤 느낌인지 모르는 상황이 있을 수 있어 지금과 같은 기획을 한것이니 해당 리스트 카테고리를 보여주는 기준에 대한 정책을 개선시키는 방향이 좋을 듯합니다.
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.
그럼 이건 다음 스프린트나 목요일 회의에서 정하나요?
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.
이건 백로그로 등록 시켜두고 스프린트 회의때 얘기해보죠
gustn99
left a comment
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.
깔끔하네요 고생하셨습니다~!
| const handleSubmitBtnClick = async () => { | ||
| const selectedCategories = Object.keys(clickedImages) | ||
| .filter((id) => clickedImages[Number(id)]) // 클릭된 이미지만 필터링 | ||
| .map((id) => styleImages.find((img) => img.id === Number(id))?.category) // category 값 가져오기 | ||
| .filter((category): category is string => !!category); // undefined 제거 | ||
|
|
||
| const requestData: Partial<PatchUserInfoRequest> = { | ||
| userStyletags: selectedCategories, | ||
| }; | ||
| console.log(requestData); | ||
|
|
||
| try { | ||
| const data = await patchUserInfoApi(requestData, currentUserId); | ||
| console.log(data); | ||
| navigate('/'); | ||
| } catch (error) { | ||
| console.error('API 요청 실패:', error); | ||
| setModalMessage('스타일 선택 중 오류가 발생했습니다.'); | ||
| console.log(requestData); | ||
| setIsModalOpen(true); | ||
| } | ||
| }; |
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.
requestData에서 userStyletags 외에 다른 데이터가 포함될 여지가 없다면 Pick을 사용하는 편이 타입 안정성 측면에서 더 좋을 것 같기는 합니다!
주요 작업 내용
기타 작업 내용
코드 리뷰 포인트
작업 화면