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

[23기 이지선] Fix: 로그인 페이지 기능 구현 수정 완료 #33

Closed
wants to merge 3 commits into from

Conversation

ddodam
Copy link

@ddodam ddodam commented Jul 15, 2021

-기존 login button 활성-비활성 반복되는 현상 수정 완료
-input 입력시 login button 활성/ input 미입력시 login button 비활성 되도록 구현 완료

wecode PR message template 및 체크 리스트 입니다.
아래 사항들을 전부 작성/체크 하시고 PR 해주세요!

수정 사항 간략한 한줄 요약

(여기에 수정 사항에 대한 간략한 한줄 요약/제목 작성해 주세요.)

  • 로그인 버튼 활성화 기능 구현(예시)

수정 사항들 자세한 내용

(여기에 수정 사항에 대한 자세한 내용을 작성해 주세요.)

  • id '@' 포함, pw 5 글자 이상 입력 시 버튼 활성화(예시)

기타 질문 및 특이 사항

(궁금한 사항들, 하면서 느낀 점들 공유해주실 것이 있으면 자유롭게 작성해 주세요.)

  • 함수 로직이 너무 긴 것 같습니다. 좀 더 효율적인 방법이 없을지 궁금합니다.(예시)

체크 리스트 (아래 사항들이 전부 체크되어야만 merge가 됩니다!)

  • 필요한 test들을 완료하였고 기능이 제대로 실행되는지 확인 하였습니다.
  • Wecode의 코드 스타일 가이드에 맞추어 코드를 작성 하였습니다.
  • 제가 의도한 파일들과 수정 사항들만 커밋이 된 것을 확인 하였습니다.
  • 본 수정 사항들을 팀원들과 사전에 상의하였고 팀원들 모두 해당 PR에 대하여 알고 있습니다.

@we-dohyeon we-dohyeon closed this Aug 12, 2021
@saengmotmi saengmotmi deleted the feature/jiseon branch November 15, 2021 04:44
@baehyunwook baehyunwook restored the feature/jiseon branch November 30, 2022 04:32
@sstaar91 sstaar91 deleted the feature/jiseon branch December 12, 2022 06:55
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.

2 participants