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

유저 인증 관련 클래스 기능 구현 & 리팩토링 #43

Merged
merged 32 commits into from
Nov 16, 2023

Conversation

sukchan-0811
Copy link
Contributor

@sukchan-0811 sukchan-0811 commented Nov 15, 2023

PR Title: 유저 인증 관련 클래스 기능 구현 & 리팩토링

Related Issue(s):

#24

PR Description:

  • HomeActivityAuthActivity 분리
  • 인증 토큰 / 유저 아이디를 캐싱 및 관리하는 SessionManager 구현
  • 인터넷 연결 관련 에러 처리
  • 로그아웃 및 회원탈퇴 API 연결
  • strings.xml 정리
  • API 호출 후 에러 처리 관련 viewmodel 전반적인 리팩토링
  • 로그인 호출 후 GET user/profile 호출, 유저 정보 저장하도록 구현 (user, userDao 등 관련 클래스 포함)
  • 게스트 모드 구현
  • 게스트 모드일 때 특정 설정 목록 비활성화 + '게스트 모드에서 나가기' 구현
  • 회원가입, 이메일 인증 시 사용자에게 조금 더 직관적이도록 UI 개선
  • 이메일 인증 화면에서 '인증번호 발송' 시 로딩 구현
Changes Included:
  • Added new feature(s)
  • Fixed identified bug(s)
  • Updated relevant documentation
Notes for Reviewer:

일단 작업이 예상보다 늦어져서 죄송합니다... PR 올리고 빠른 리뷰를 받는 게 나을 것 같아서 일단 올렸는데 테스트 코드 작성도 최대한 빨리 해보도록 하겠습니다...!

Reviewer Checklist:

  • Code is written in clean, maintainable, and idiomatic form.
  • Automated test coverage is adequate.
  • All existing tests pass.
  • Manual testing has been performed to ensure the PR works as expected. (-->Some issues are being handled in another branch)
  • Code review comments have been addressed or clarified.

Additional Comments:

Add any other comments or information that might be useful for the
review process.

@sukchan-0811 sukchan-0811 added frontend Feature implementation for frontend user-interface Issues related to UI/UX labels Nov 15, 2023
@sukchan-0811 sukchan-0811 self-assigned this Nov 15, 2023
Copy link
Contributor

@yjeong-k yjeong-k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

늦은 시간까지 정말 너무너무 고생많으셨습니다...!!!😭 죄송하실 것 전혀 없어요!!!
코드 정성스레 잘 짜주시고 리팩토링 꼼꼼하게 잘해주셔서 정말 감사해요!!
코드 쭉 보면서 사소한 것들은 suggestion으로 남겼고 논의사항 두 가지만 아래와 같이 남겨둡니다..!

  1. 로그인, 이메일 인증 등에서는 back click을 일일이 다 빼주셨는데 settings 화면들에는 그냥 두셔서 혹시 여기는 따로 남겨두신 이유가 있을까요?? (빼달라는 거 아니고 진짜로 궁금해서 여쭤봅니닷ㅎㅎㅎ)
  2. 로그인 하고 앱을 껐다 다시 키면 자동 로그인이 되는데, 이 때 설정에 들어가보면 로그인 모드가 아닌 게스트모드의 설정이 보입니다..! 진짜 게스트로 인식하는건지 설정화면만 그렇게 보이는지는 모르겠지만,,,

@89645321
Copy link
Contributor

늦은 시간까지 정말 너무너무 고생많으셨습니다...!!!😭 죄송하실 것 전혀 없어요!!! 코드 정성스레 잘 짜주시고 리팩토링 꼼꼼하게 잘해주셔서 정말 감사해요!! 코드 쭉 보면서 사소한 것들은 suggestion으로 남겼고 논의사항 두 가지만 아래와 같이 남겨둡니다..!

  1. 로그인, 이메일 인증 등에서는 back click을 일일이 다 빼주셨는데 settings 화면들에는 그냥 두셔서 혹시 여기는 따로 남겨두신 이유가 있을까요?? (빼달라는 거 아니고 진짜로 궁금해서 여쭤봅니닷ㅎㅎㅎ)
  2. 로그인 하고 앱을 껐다 다시 키면 자동 로그인이 되는데, 이 때 설정에 들어가보면 로그인 모드가 아닌 게스트모드의 설정이 보입니다..! 진짜 게스트로 인식하는건지 설정화면만 그렇게 보이는지는 모르겠지만,,,

세팅이랑 인증 관련된 부분이랑 너무 연결이 되어있어서 코멘트 남기신 것 같은데, 세팅 쪽은 다른 브랜치로 pr 올릴 예정입니다!

@yjeong-k
Copy link
Contributor

@89645321

늦은 시간까지 정말 너무너무 고생많으셨습니다...!!!😭 죄송하실 것 전혀 없어요!!! 코드 정성스레 잘 짜주시고 리팩토링 꼼꼼하게 잘해주셔서 정말 감사해요!! 코드 쭉 보면서 사소한 것들은 suggestion으로 남겼고 논의사항 두 가지만 아래와 같이 남겨둡니다..!

  1. 로그인, 이메일 인증 등에서는 back click을 일일이 다 빼주셨는데 settings 화면들에는 그냥 두셔서 혹시 여기는 따로 남겨두신 이유가 있을까요?? (빼달라는 거 아니고 진짜로 궁금해서 여쭤봅니닷ㅎㅎㅎ)
  2. 로그인 하고 앱을 껐다 다시 키면 자동 로그인이 되는데, 이 때 설정에 들어가보면 로그인 모드가 아닌 게스트모드의 설정이 보입니다..! 진짜 게스트로 인식하는건지 설정화면만 그렇게 보이는지는 모르겠지만,,,

세팅이랑 인증 관련된 부분이랑 너무 연결이 되어있어서 코멘트 남기신 것 같은데, 세팅 쪽은 다른 브랜치로 pr 올릴 예정입니다!

앗그렇군요 감사합니다!!! 그러면 위 1, 2번 모두 그 다른 세팅관련 브랜치에서 해결될 부분들일까요??!

@89645321
Copy link
Contributor

@89645321

늦은 시간까지 정말 너무너무 고생많으셨습니다...!!!😭 죄송하실 것 전혀 없어요!!! 코드 정성스레 잘 짜주시고 리팩토링 꼼꼼하게 잘해주셔서 정말 감사해요!! 코드 쭉 보면서 사소한 것들은 suggestion으로 남겼고 논의사항 두 가지만 아래와 같이 남겨둡니다..!

  1. 로그인, 이메일 인증 등에서는 back click을 일일이 다 빼주셨는데 settings 화면들에는 그냥 두셔서 혹시 여기는 따로 남겨두신 이유가 있을까요?? (빼달라는 거 아니고 진짜로 궁금해서 여쭤봅니닷ㅎㅎㅎ)
  2. 로그인 하고 앱을 껐다 다시 키면 자동 로그인이 되는데, 이 때 설정에 들어가보면 로그인 모드가 아닌 게스트모드의 설정이 보입니다..! 진짜 게스트로 인식하는건지 설정화면만 그렇게 보이는지는 모르겠지만,,,

세팅이랑 인증 관련된 부분이랑 너무 연결이 되어있어서 코멘트 남기신 것 같은데, 세팅 쪽은 다른 브랜치로 pr 올릴 예정입니다!

앗그렇군요 감사합니다!!! 그러면 위 1, 2번 모두 그 다른 세팅관련 브랜치에서 해결될 부분들일까요??!

넵!!

Copy link
Contributor

@yjeong-k yjeong-k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

감사합니다!😊

@yjeong-k yjeong-k merged commit 6b455a9 into main Nov 16, 2023
1 check passed
@yjeong-k yjeong-k deleted the refactor/auth-clean-ver branch November 16, 2023 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Feature implementation for frontend user-interface Issues related to UI/UX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants