Skip to content
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

[에디] 회원가입 미션 3, 4단계 제출합니다. #23

Open
wants to merge 16 commits into
base: junjange
Choose a base branch
from

Conversation

junjange
Copy link
Member

췤췤 이번에도 잘 부탁드립니다~~ 🫡

구현 사항

  • 회원가입 뷰에 유효성 검사 로직을 추가한다.
  • 유효성 검사 로직에 대한 테스트 코드를 추가한다.
  • 디자인 시안을 참고하여 모든 필드가 에러 없이 채워진 경우에만 Sign up 버튼을 활성화한다.
  • 유효성 검사 로직과 뷰 로직을 나누어 관심사를 분리한다.
  • 모든 로직에 테스트 코드를 추가한다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant