Merged
Conversation
왜? 이번 버전에 구글로그인이 생길 예정이라 네이밍을 확실하게 이메일 로그인으로 변경하여 구분짓기 위함!
왜? 기존의 온보딩 페이지가 온보딩 역할을 하지 못하고 있어서 온보딩 로직을 제거함 온보딩 페이지는 조금 수정하여 로그인 페이지로 사용할 예정임 따라서 최초 설치시 온보딩 페이지로 이동시키는 로직과 관련한 코드를 모두 제거함
왜? 이번 버전에 구글로그인이 기능이 들어가면서 이제 이 페이지는 온보딩 역할이 아니라 로그인을 하는 페이지로 사용하려고 함!
왜? as-is) 이메일 로그인 페이지가 시작페이지였음 to-be) 로그인 페이지가 시작페이지로! (성격이 다름)
새로운 디자인에 맞게 변경함!
어떻게? 가입 경로 선택 뷰를 클릭하면 바텀시트를 띄우고 선택한 값을 저장하여 화면에 반영하는 로직을 구현함!
왜? 작은 기기에서 1줄을 넘어가는 경우가 발생할 수 있다고 판단하여 해당 속성을 추가함
왜? 마케팅 동의 약관인데 아이디를 잘못 설정하였음
어떻게? 체크박스의 clickable을 false로 두고 약관 항목을 리니어 레이아웃으로 감쌌음 터치 범위를 늘리기 위해서도 있고 checkedChangeListener 때문에 상위 레벨 항목의 check state를 변경할 때 의도치 않은 동작을 하는 것을 막기 위해서임
왜? 넥룸 최종 가입을 위해선 추가적으로 입력해야 할 정보들이 있다. 구글 인증을 완료하면 서버에서 주는 isComplete 값에 따라 정보를 입력 하고 최종적으로 회원 가입을 완료 하기 위한 페이지로 이동한다.
왜? 가입이 완료 되었을 때 회원가입 페이지에서 완료되었다고 결과를 돌려주고 이전 페이지에서 테마 선택페이지로 넘어가도록 하기 위함
왜? 기존에는 사용자 정보를 저장하면서 그와 동시에 로그인 했다는 값을 true로 설정하고 있었다. 하지만 이것엔.. 문제가 있다! 1. saveAdminInfo라는 함수명만 보고 loggedIn을 저장할 것이라는 동작을 예상을 할 수 없다. 2. 로그인은 하지 않고 사용자 정보만 저장하는 경우가 새로 생겼다. 그래서 로그인 상태를 저장하는 setLoggedIn 라는 함수를 따로 만들었다.
왜? 로그인 설정을 하고 난 다음 토큰을 저장했더니 그새 그사이에 api 호출 하는 로직이 중간에 끼어있어서 토큰 만료됐다고 로그아웃 되어버림 그래서 토큰을 먼저 저장하고 로그인 설정을 하도록 함
왜? 가입 이유는 선택 항목이므로 선택한 값이 없다면 서버에 빈칸으로 보내면 됨 requireNotNull을 하고 있어서 값을 지정하지 않으면 가입이 안되는 이슈가 있었음
왜? 작은 스마트폰 사이즈를 대응하기 위해 스크롤 뷰를 추가함
왜? loginState를 collect해서 화면을 이동시켜주는 로직이랑 중복되어 삭제함
왜? 화면 이동을 안전하게 하기 위해 safeNavigate 사용
왜? 가이드 보기 팝업과 보여지는 타이밍이 동일해 겹치기도 하고 비밀번호 설정이 정말 필요할 때 요청하는 게 처음 설치하여 진입한 사용자들에게 스트레스를 덜줄것같음 onResume에 두니 과도하게 많이 요청함
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
주요 작업 목록
자세한 내용은 각 커밋에 상세히 작성해둠!