[Refactor] SP4 탈퇴 기능 및 디자인 QA 반영 #612
Merged
Conversation
This file contains hidden or 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
hansoojeongsj
requested review from
KIMGEONHWI,
constantly-dev,
heesunee and
rtttr1
as code owners
|
✅ Storybook 배포 완료! 🔗 https://67e4fd1fd2c7078dceec04a4-grikfqwcst.chromatic.com/ |
constantly-dev
approved these changes
Nov 30, 2025
Member
constantly-dev
left a comment
constantly-dev
left a comment
There was a problem hiding this comment.
수고하셨습니다~~ 디자인 수정이 많아서 크게 없는 것 같아요!
Comment on lines
+15
to
+17
| const { data, isError, isPending } = useGetDancerDetail(id ?? '', { | ||
| enabled: Boolean(id), | ||
| }); |
Member
There was a problem hiding this comment.
useGetDancerDetail에 id를 넘겨서 내부에서 enabled를 지정하는게 아니라 외부에서 지정해준 이유는 명시적으로 id가 있을 때 query 요청이 간다는 것을 보여주기 위함인가요??
(의도에 대한 질문이라 정답은 없을 것 같아요!)
Member
Author
There was a problem hiding this comment.
넵 id가 있을 때만 요청이 나간다는 걸 드러내는게 명확하다고 느꼈고,
기존 코드에서 eslint-ignore를 쓰는 구조를 없애고 싶어서 enabled로 제어하도록 수정했습니다.
KIMGEONHWI
approved these changes
Dec 1, 2025
Collaborator
KIMGEONHWI
left a comment
KIMGEONHWI
left a comment
There was a problem hiding this comment.
LGTM!! 꼼꼼 수정님 감사합니다.
Deploying dash-client-dev with
|
| Latest commit: |
f818dc1
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://e0f458ae.dash-client-dev.pages.dev |
| Branch Preview URL: | https://refactor--610-sp4-qa.dash-client-dev.pages.dev |
Deploying dash-client with
|
| Latest commit: |
f818dc1
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://b9f3bce4.dash-client.pages.dev |
| Branch Preview URL: | https://refactor--610-sp4-qa.dash-client.pages.dev |
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.
📌 Related Issues
✅ 체크 리스트
📄 Tasks
디자인 QA
전화번호 언더바를 없애주세요 (모바일 크롬에서 보임)
여기 왜 언더바가 두개나 뜨죠?
핸드폰에서 크롬으로
www.da-sh.kr에 접속했을때, 전화번호, 장소, 계좌번호 등 모든 자동 링크를 제거해달라는 요청이 있어서index.html에<meta name="format-detection" content="no" />을 추가해줬어요. 인터넷 서치한 후, 추가했는데http://192.xxx.xxx.xxx:5173/로 핸드폰 크롬으로 확인했는데 언더라인이 두개로 보이지 않았어요. 해당 문제는 배포주소에서 볼 수 있는 문제 같아서.. 배포 후 추가 확인이 필요한 부분이에요.프로필 기본 이미지가 좀 작은 것 같기도…? 원 안에 안 맞네요?
ic_profile_basic.svg 자체에 여백이 있던 문제라, 아이콘만 변경해주었어요.
댄서네임이 medium이 아니라, semibold입니다.
text의 tag="b1_sb" 변경해주었어요.
혼자 작업하다 발견했는데
/dancer/:id에서 장르 칩이 많이 나오는 경우, 배치가 깨지고 있어서 해당 부분 수정했어요.-> 변경사항이 있는 파일들에서는 sprinkles 제거했고 애스터리스크로 적혀있던 스타일 임포트도 수정해주었어요.
기능 QA
탈퇴 후 재가입 성공시, finishStep 문구 변경
useLoginMutation에서 서버에서 내려오는isDeleted값을 전달하도록 수정했어요.useLoginMutation에서 로그인 성공 시onSuccess({{ ..., isDeleted}})를 넘기고Onboarding.tsx에서const isDeleted = tokenRef.current?.isDeleted ?? false로 값을 보관해요.그리고
FinishStep에서isDeleted여부에 따라 문구 분기 처리해주었어요.온보딩을 하던 중,
OnboardingHeader의 버튼을 누르면submit처리가 되어서 onboard api가 보내지고 있었어요. 그래서 각각 버튼에type="button"추가해주었어요.또한, 온보딩에서 정보 입력하는 단계에서만 뒤로가기가 보여지는데, 해당 화면의
(-1)을 해줘서 뒤로가기를 진행하게 되면, 카카오로그인 redirect로 이동하고 있었어요. 그래서 해당 부분 수정해서,뒤로가기 -> login,로고 -> home으로 페이지 이동하게 수정했어요.그래서, 이제 탈퇴 이후 온보딩까지 완벽히 수행한 재가입자만
login->verify->onboard->진짜 가입플로우가 잘 진행되고 있어요!기능 QA 추가작업 필요함 !!
⭐ PR Point
📷 Screenshot
🔔 ETC