관리자 코드를 제거하고 앱 비밀번호를 사용하여 인증하는 방식 구현 #99
Merged
Conversation
This file contains 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
왜? as-is 기준으로 게임을 바로 시작하는게 아니라 관리자 코드를 입력하러 이동하기 때문에 적절한 네이밍이 아니다. 또한 to-be 기준으로 비밀번호 체크를 하러 이동할것이기 때문에 테마가 클릭 되었을 때 호출되는 함수 정도로 네이밍을 변경한다.
왜? 비밀번호 설정 후 호출했던 화면으로 돌아가서 이어서 어떠한 액션을 하는데 그때 필요한 값을 resultData라고 정했다. 예를 들어 테마 선택 후 비밀번호 체크하고 게임을 시작할 때 테마 아이디가 필요하다. 이 값을 비밀번호 체크 화면으로 넘겨주었다가 다시 돌려받는 것이다.
기존에는 관리자 코드를 체크하는 화면으로 넘어갔지만 비밀번호 체크하는 화면으로 넘어갔다가 비밀번호가 일치하면 게임 시작하는 화면으로 이동하게 바꾼다.
왜? 더이상 관리자 코드를 사용하지 않는다.
왜? 게임 시작을 하려면 앱 비밀번호가 필요하므로 미설정된 사용자들 대상으로 테마 선텍페이지 진입시 비밀번호 유무에 따라 비밀번호 설정 유도 팝업을 띄운다. 그리고 혹시나 닫기를 누를 사용자들이 있을 수 있으므로 테마 클릭시에도 똑같이 띄운다.
왜? start라는 함수가 레거시 코드라서 제거하고 tryGameStart에 포함시킴
왜? 뒷배경이 보이는 로딩을 노출하니 화면이 빠르게 이동하는 모습이 노출되어 어색하게 보임 그래서 붙투명한 로딩을 추가함.
활동 유지 안함을 켜놓고 다이얼로그가 떠있는 상태에서 백그라운드로 나갔다가 재진입시 다이얼로그가 중복으로 뜨는 이슈가 있으니 SingleTop으로 화면을 띄운다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
as-is)
게임을 시작하려면 관리자 코드를 입력해야 했음
to-be)
게임을 시작하려면 앱 비밀번호를 입력해야 함.
테마 선택 페이지랑 테마를 선택했을 때 만약 로컬에 저장되어있는 앱 비밀번호가 없으면
비밀번호 설정하는 페이지로 이동하게 해두었음
마이 페이지에서는 비밀번호를 변경 가능하게 했음
관리자 코드 관련한 코드는 전부 삭제함