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

Feature/회원가입 기능 이슈 해결 #90

Merged
merged 2 commits into from
Aug 6, 2023
Merged

Conversation

ChoiWonBeen
Copy link
Member

[#39, #83] request

  • 회원가입 아이디 중복확인 중, 발생하는 에러메시지를 서버에서 받아 출력하게 수정했습니다.
  • 회원가입 마무리에 닉네임 변경 기능을 추가했습니다.
    • 아직 서버에서 회원가입 api에 임시 accessToken을 제공하고 있지 않아 추가되는대로 닉네임변경이 가능해지도록 수정할 예정입니다. 그 외 닉네임 변경의 예외처리나, api호출까지는 작성해두었습니다.

Please check if the PR fulfills these requirements

  • It's submitted to develop branch, not the main branch
  • The commit message follows our guidelines
  • There are no warning message when you run yarn lint
  • Docs updated for breaking changes

Screenshot

image image

Copy link
Contributor

@KimKyungYun KimKyungYun left a comment

Choose a reason for hiding this comment

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

수정내용 확인했습니다. 고생하셨습니다!

@ChoiWonBeen ChoiWonBeen merged commit 1fb8b53 into develop Aug 6, 2023
1 check passed
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