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/#78 디자인 수정 적용 & UI 관련 오류들 수정 #79

Merged
merged 12 commits into from
Jul 21, 2023

Conversation

ChoiWonBeen
Copy link
Member

[#78] request

전체적으로 디자인 바뀐 부분, 변경 중 에러난 부분들을 찾아 일차적으로 수정했습니다.

  • 자동로그인 체크박스가 탈퇴 UI에 포함된 CSS랑 동일해진 문제 수정
  • 검색창 제목 글 안맞는 문제 수정
  • 아이디 / 비밀번호 찾기 페이지 높이가 낮은 디바이스에서 위아래가 잘리는 문제 수정
  • 메인페이지 공용 상단바에 아이콘 추가 / 링크 수정
  • 아이디찾기 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

코드리뷰가 필요한 부분은 아니고, 기능 동작 한번씩 해보면서 문제없는지와, 운영체제(윈도우)에서 오류없는지만 봐주시면 감사하겠습니당

이것도 근데 몇 분 안돼서 찾은 부분들인데, 한 번 기능 쭉 돌려보면서 문제있는 부분들은 찾아서 이슈에 담아두고 수정해야할 것 같아요.
아이디찾기 - 비밀번호 찾기 PC버전 UI도 추가됐으니 회원가입때 썼던 UI 처럼 활용해 만들면 되겠네요

@ChoiWonBeen ChoiWonBeen removed the request for review from jaewoogwak June 23, 2023 03:26
@ChoiWonBeen ChoiWonBeen changed the title Feature/#78 Feature/#78 디자인 수정 적용 & UI 관련 오류들 수정 Jun 23, 2023
Copy link
Contributor

@chaeseungyun chaeseungyun left a 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을 반환하던데 무슨 의미인지 잘 모르겠어요
그리고 비밀번호 찾기 마지막 페이지에 인증번호 보내기가 아니고 변경 완료와 같은 문구가 있어야 할 것 같습니다

@ChoiWonBeen
Copy link
Member Author

@chaeseungyun 비밀번호 찾기 UI와 API 수정해서 정상작동합니다!
제공하는 accessToken을 활용해 내 정보 변경 api(PATCH: /user/me) 를 호출하게끔 바뀌었네요

추후 회원가입단에서 nickname 수정하는 함수도 같이 활용할 예정
src/api/user/index.ts Outdated Show resolved Hide resolved
@chaeseungyun chaeseungyun self-requested a review July 14, 2023 04:00
@chaeseungyun
Copy link
Contributor

고생하셨습니다!!

@ChoiWonBeen ChoiWonBeen merged commit 3425801 into develop Jul 21, 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