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.

Sign up for GitHub

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

Jump to bottom

[Fix] #87 - 회원가입 및 비밀번호 변경 뷰 1차 QA 반영 #89

Merged
merged 7 commits into from
Jan 7, 2023
Merged

[Fix] #87 - 회원가입 및 비밀번호 변경 뷰 1차 QA 반영 #89

merged 7 commits into from
Jan 7, 2023

Conversation

lsj8706
Copy link
Member

@lsj8706 lsj8706 commented Jan 7, 2023

🌴 PR 요약

🌱 작업한 브랜치

🌱 PR Point

  • 회원 가입 완료 후 메인페이지로 화면 전환
  • 이메일 중복일 때 텍스트 안내
  • 닉네임, 이메일 텍스트 필드 변경 시 가입하기 버튼 비활성화
  • 비밀번호 변경뷰에서 비밀번호 텍스트 제거

📌 참고 사항

  • qa 43번 사항인 회원가입의 비밀번호 입력 시 @ 특수문자 입력하면 다음 텍스트 필드로 넘어가는 문제 해결 은 슬랙에 질문 올린거 답변 달리면 따로 이슈 파서 해결하겠습니다.

📸 스크린샷

기능 스크린샷
기능이름 스크린샷 첨부

📮 관련 이슈

@lsj8706 lsj8706 added Fix 문제 해결, 코드 수정 세진🤝🏻 labels Jan 7, 2023
@lsj8706 lsj8706 self-assigned this Jan 7, 2023
L-j-h-c
L-j-h-c approved these changes Jan 7, 2023
View reviewed changes
Copy link
Contributor

@L-j-h-c L-j-h-c left a comment

Choose a reason for hiding this comment

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

수고하셨슴다~

@lsj8706 lsj8706 merged commit 40b64ab into sopt-makers:develop Jan 7, 2023
@lsj8706 lsj8706 deleted the fix/#87-회원가입-비밀번호변경뷰-QA branch January 7, 2023 16:27
devxsby
devxsby reviewed Jan 9, 2023
View reviewed changes
Copy link
Member

@devxsby devxsby left a comment

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
Reviewers

@devxsby devxsby devxsby left review comments

@L-j-h-c L-j-h-c L-j-h-c approved these changes

@yangsubinn yangsubinn Awaiting requested review from yangsubinn

Assignees

@lsj8706 lsj8706

Labels
Fix 문제 해결, 코드 수정 size/L 세진🤝🏻
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Fix] 회원가입 및 비밀번호 변경 뷰 1차 QA
3 participants
@lsj8706 @L-j-h-c @devxsby