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.

Sign up for GitHub

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

Jump to bottom

[App] 로그인 로직 버그 수정 #130

Open
TaerryKim opened this issue Sep 16, 2018 · 0 comments
Open

[App] 로그인 로직 버그 수정 #130

TaerryKim opened this issue Sep 16, 2018 · 0 comments

Comments

@TaerryKim
Copy link
Member

로그인 시 로직 수정 필요

  • 현재 로그인 요청시 서버로부터 바로 등록되고, 닉네임이 없을 경우 가입을 시도하는 기형적인 구조.

  • 로그인 시 아이디가 없으면 null 값을 리턴하고, 가입 페이지로 이동시키며, 닉네임을 설저하면 가입 요청하도록 로직 변경
TaerryKim added a commit that referenced this issue Sep 17, 2018
@TaerryKim
#130 로그인 로직 수정
6fdec9b 
* 현재 로그인 요청시 서버로부터 바로 등록되고, 닉네임이 없을 경우 가입을 시도하는 기형적인 구조.

* 로그인 시 아이디가 없으면 null 값을 리턴하고, 가입 페이지로 이동시키며, 닉네임을 설저하면 가입 요청하도록 로직 변경

* http error status code에 대한 대응 로직 추가

* 로그인 정보등을 sharedPreferences 로 관리

* 코드 리펙토링 수행
  - 불필요한 코드 제거
KimJuhyun92 added a commit that referenced this issue Sep 17, 2018
@KimJuhyun92
Merge pull request #134 from HanRiverMeetup/issue-130
5a036a2 
#130 로그인 로직 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@TaerryKim