Open
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
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.
📝 개요
회원 탈퇴 UX를 피그마 스펙에 맞춰 구현했습니다. 마이페이지에서 탈퇴 사유·상세 입력 후 제출하면 탈퇴 완료 안내 모달이 뜨고, 「확인」 시 세션을 비우고 홈(/)으로 이동합니다. 모달 패널 레이아웃·타이포·색 토큰을 디자인 시스템(text-heading-base, text-body-base, 오버레이 bg-black/70 등)에 맞췄고, 필요한 경우 tsconfig.json 경로/tsconfig 정리도 포함되어 있습니다.
🔗 관련 이슈
🛠️ 변경 사항 (Checklist)
✅ 아래 내용을 한 번 더 점검해 주세요
1. 의도와 가독성 (Naming & Readability)
2. 타입과 논리 (Type Safety & Logic)
any사용을 지양하고, 모든 함수의 반환 타입을 명시했나요?null/undefined), 에러가 발생할 경우를 처리했나요?3. 코드 다이어트 (Clean-up)
console.log나 사용하지 않는import를 모두 지웠나요?4. 지속 가능성 (Sustainability)
README업데이트가 필요한가요?💭 회고 (Optional)
피그마와 스펙이 여러 차례 갱신되어 dropcomplete 여백(16px 간격 vs 31px 등)을 최종 프레임에 맞춰 정리했습니다.
ESLint set-state-in-effect 이슈는 탈퇴 모달 onOpenChange에서 완료 모달 상태 초기화하도록 회피했습니다.
Textarea 공통 수정 대신 탈퇴 모달에서만 비활성 토큰을 덮는 방향으로 팀 규칙을 맞췄습니다.