-
Notifications
You must be signed in to change notification settings - Fork 0
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/#78 디자인 수정 적용 & UI 관련 오류들 수정 #79
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지금 확인해보니 비밀번호 찾기 관련 api에 변경이 있었던 것 같습니다.
비밀번호 변경 관련 api(PATCH 'user/'password')가 사라진 상태라 비밀번호 찾기가 정상적으로 이루어지지 않고있네요
현재 비밀번호 찾기 api를 호출하면 accessToken을 반환하던데 무슨 의미인지 잘 모르겠어요
그리고 비밀번호 찾기 마지막 페이지에 인증번호 보내기가 아니고 변경 완료와 같은 문구가 있어야 할 것 같습니다
@chaeseungyun 비밀번호 찾기 UI와 API 수정해서 정상작동합니다! |
추후 회원가입단에서 nickname 수정하는 함수도 같이 활용할 예정
고생하셨습니다!! |
[#78] request
전체적으로 디자인 바뀐 부분, 변경 중 에러난 부분들을 찾아 일차적으로 수정했습니다.
Please check if the PR fulfills these requirements
develop
branch, not themain
branchyarn lint
코드리뷰가 필요한 부분은 아니고, 기능 동작 한번씩 해보면서 문제없는지와, 운영체제(윈도우)에서 오류없는지만 봐주시면 감사하겠습니당
이것도 근데 몇 분 안돼서 찾은 부분들인데, 한 번 기능 쭉 돌려보면서 문제있는 부분들은 찾아서 이슈에 담아두고 수정해야할 것 같아요.
아이디찾기 - 비밀번호 찾기 PC버전 UI도 추가됐으니 회원가입때 썼던 UI 처럼 활용해 만들면 되겠네요