Skip to content

Conversation

@MoonsuKang
Copy link
Member

@MoonsuKang MoonsuKang commented Feb 20, 2025

Related issue 🛠

closed #166

어떤 변경사항이 있었나요?

  • 🐞 BugFix Something isn't working
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (Junit, etc.)

CheckPoint ✅

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 컨벤션에 맞게 작성했습니다. (필수)
  • merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
  • BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

Work Description ✏️

  • 닉네임 벨리데이션 검증 개선
  • 회원가입 실패 시 다이얼로그 띄우기

Uncompleted Tasks 😅

  • 피커는 이제 안할래 진짜 싫다

To Reviewers 📢

  • 대략 2시간 삽질 했는데 피커문제를 해결하지 못했습니다.. 왜 페이지를 이동하면 OnValueChage를 호출하는지 모르겠습니다.
    SaveStateHandle 문젠가 싶어 갈아버리고 했으나 이또한 아니었습니다. YearMonthPicker에서 LaunchedEffect문제 같은데... 흠...
    과열상태라 잠깐 자고 일어나서 해야될 것 같습니다 하...

@DongChyeon
Copy link
Member

DongChyeon commented Feb 21, 2025

[p1]
BirthDay -> TimeOfBirth 화면으로 이동할 때
실기기에서는 updateBirthDate를 호출하여 생년월일 값이 잘못 나옵니다.

이를 방지하기 위해 생년월일 입력 화면이 아닐 경우 updateBirthDate 실행되지 않도록 수정되도록 했으니 확인 부탁드립니다.

Copy link
Member

@DongChyeon DongChyeon left a comment

Choose a reason for hiding this comment

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

난 내 코드 믿어.

@MoonsuKang MoonsuKang merged commit 732f613 into develop Feb 21, 2025
1 check passed
@DongChyeon DongChyeon deleted the refactor/#166-nickname-picker branch June 10, 2025 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 닉네임 정규표현식을 수정하고 피커 오류를 해결합니다

3 participants