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

Bw 137 android make edit privacy activity #49

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

Conversation

Unique0902
Copy link
Collaborator

개요

PR에 관한 개략적인 설명을 써주세요

개인 정보 수정화면 작성

수정사항

작업 내용을 써주세요

개인 정보 수정화면 layout 작성
mypage fragment에서 이동되게 연결
닉네임, 이메일 정보 받아 수정화면에서 띄움
비밀번호 수정하기 버튼 클릭시 비밀번호 입력란 띄우는 interaction 추가

유의사항

리뷰 시 참고할 내용을 써주세요

앞으로 보완해야할 사항입니다 ------------------------

mypage에서 이름 정보도 같이 받아서 넘겨야할것같은데 mypage api에서 이름은 받아오지 않는것 같은데 이름도 같이 받아올수있게 수정할 필요있음

비밀번호 수정할때 비밀번호 유효성 검사 + 유효성에따른 버튼 상태 조정

비밀번호 수정 api 필요

후에 프로필 이미지 관련 인터렉션 추가

프로필 수정 api 필요 (아마 이메일 변경은 안되고 이름, 닉네임 정도 수정하는 걸로)

이름 닉네임 수정할수있게 textinput 이용한 상호작용 필요 (지금은 textview로 구현되있음)

버튼 보니깐 비밀번호 수정버튼만 있던데 이름이나 닉네임 수정할때는 각 수정 페이지를 만들어 하나씩 수정하게 하는게 나을것같기도함 (프로필수정 api도 그러면 닉네임 수정 api, 이름 수정 api 이런식으로 따로 만들면 될듯)

비밀번호 수정 전에 보통 현재 비밀번호 입력해서 보안검사하던데 우리도 이런식으로 해야할듯요

@Unique0902 Unique0902 requested a review from nonaninona August 27, 2023 13:12
@nonaninona
Copy link
Contributor

보완사항에서 출시까지 꼭 필요한 사항이랑
하면 좋은 거(아마 안할 것 ^^)을 구분을 좀 해보자면

꼭 필요

mypage에서 이름 정보도 같이 받아서 넘겨야할것같은데 mypage api에서 이름은 받아오지 않는것 같은데 이름도 같이 받아올수있게 수정할 필요있음
비밀번호 수정 api 필요
비밀번호 수정할때 비밀번호 유효성 검사 + 유효성에따른 버튼 상태 조정

하면 좋은 거

후에 프로필 이미지 관련 인터렉션 추가
프로필 수정 api 필요 (아마 이메일 변경은 안되고 이름, 닉네임 정도 수정하는 걸로)
이름 닉네임 수정할수있게 textinput 이용한 상호작용 필요 (지금은 textview로 구현되있음)
비밀번호 수정 전에 보통 현재 비밀번호 입력해서 보안검사하던데 우리도 이런식으로 해야할듯요

인 것 같은데 어떻게 생각하시나욤?

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