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

마이페이지,계정관리 fix #102

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

rktguswjd
Copy link
Member

@rktguswjd rktguswjd commented Jun 26, 2021

📌 마이페이지 수정사항

  • 참여 테스트 변수명 변경되어 출력 안됨 -> 변경된 변수명 적용하여 해결
  • 마이페이지 깜밖임 -> 데이터 초기화하라구요(?)

📌 계정관리 수정사항

  • 프로필 변경 후 변경된 이미지 출력 -> 출력됨 / 무엇이 좋을까요
  • 현재 적용 : 이미지 업로드 후 업로드된 url를 받아서 변수에 담아 미리보기 형식으로 출력
  • 이미지 업로드 -> 업로드된 url 받아서 바로 프로필로 적용 -> 적용이 완료 된 후 user정보를 update (spinner 필요할 듯 (?))
  • 닉네임 변경 되어도 입력된 값이 그대로 남아있음 -> 닉네임 변경 후 user정보를 한번 더 업데이트 시켜줌.
  • user 닉네임이 input의 placeholder로 들어가 있음 (placeholder가 아닌 value(?)로 띄워주면 사용자가 지우고 새로운 닉네임을 쳐야하기때문).
  • 닉네임을 placeholder로 띄워주기 때문에 user정보를 한 번더 업데이트함. 업데이트 하는 동안 spinner 처리

✔ 관련 이슈 #92

@rktguswjd rktguswjd added the mypage My page label Jun 26, 2021
@rktguswjd rktguswjd linked an issue Jun 26, 2021 that may be closed by this pull request
10 tasks
@rktguswjd rktguswjd self-assigned this Jun 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mypage My page
Projects
None yet
Development

Successfully merging this pull request may close these issues.

마이페이지 추가 기능
1 participant