Skip to content

Conversation

@suhyeon0111
Copy link
Contributor

Pull Request

관련 이슈

작업 내용

  • 모든 항목이 정상적으로 입력이 됐음에도 하단 다음 버튼 disabled 해제가 안되던 문제 해결
  • 닉네임 api 호출 후 사용가능한 닉네임이면 하단에 caption으로 안내 기능 추가
  • 누락된 에러 코드 추가
  • 마이페이지 카메라 버튼 배경 색상 - 디자인 토큰 값으로 변경

참고 사항

체크리스트

  • 기능이 정상 동작하는지 확인
  • 로컬 빌드/스토리북/테스트 통과
  • 불필요한 코드/주석 제거

@suhyeon0111 suhyeon0111 self-assigned this Jan 3, 2026
@suhyeon0111 suhyeon0111 added the 🐞 bug 기능 오류, 화면 깨짐, 예외 발생 등 예상치 못한 문제 label Jan 3, 2026
@auto-assign auto-assign bot requested review from DevHyungJun and wlrnjs January 3, 2026 08:30
@vercel
Copy link

vercel bot commented Jan 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
fmi-fe Ready Ready Preview, Comment Jan 3, 2026 8:36am

@wlrnjs
Copy link
Member

wlrnjs commented Jan 3, 2026

소셜 로그인 로직도 같이 올라온것 같은데 맞을까요?

@github-actions
Copy link

github-actions bot commented Jan 3, 2026

Test Results

314 tests   314 ✅  27s ⏱️
 61 suites    0 💤
  1 files      0 ❌

Results for commit df7b50d.

Copy link
Member

@wlrnjs wlrnjs left a comment

Choose a reason for hiding this comment

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

감사합니다~~


const isNextEnabled = isValid && isEmailAuthVerified && isNicknameVerified;

console.log("nickname>>> ", isNicknameVerified);
Copy link
Member

Choose a reason for hiding this comment

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

콘솔은 주석이나 삭제해서 올려주세요

onError: (error) => handlerApiError(EMAIL_CHECK_CODE_MESSAGE, error.code),
onError: (error) => {
handlerApiError(EMAIL_CHECK_CODE_MESSAGE, error.code);
console.log("error>> ", error);
Copy link
Member

Choose a reason for hiding this comment

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

콘솔은 주석이나 삭제해서 올려주세요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 bug 기능 오류, 화면 깨짐, 예외 발생 등 예상치 못한 문제

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants