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

[Feat] 로그인 중앙화 Mock 서버 연동 #1028

Open
wants to merge 45 commits into
base: feature/login-main
Choose a base branch
from

Conversation

leeeyubin
Copy link
Member

@leeeyubin leeeyubin commented Jan 16, 2025

What is this issue?

로그인 중앙화를 위한 Mock 서버를 연동합니다.

  • :data:auth 모듈, :domain:auth 모듈 생성
  • mock 서버를 위한 Retrofit 모듈 생성
  • 기존의 모든 UI 네비게이션으로 연결
  • 기존 UI 수정 (안 쓰는 뷰 삭제, 소셜계정연동 뷰 생성 등)
  • 새로운 피처 구현 (타이머 로직, 화면 별 AuthStatus 지정 등)
  • 서버통신 구현 (핸드폰 인증 생성, 핸드폰 인증 검사, 로그인, 회원가입)
  • 전화번호 포멧 구현 (by @l2hyunwoo )

Screenshot

소셜계정연동 화면
SocialAccountScreen
전화번호 포멧
CertificationScreen
타이머 일반 로직
CertificationScreen
타이머 에러 상황(타임아웃)
CertificationScreen
Screen_Recording_20250117_063313_SOPT.DEBUG.mp4
Screen_Recording_20250117_063617_SOPT.DEBUG.mp4
소셜계정찾기 뷰
AuthMainScreen
Screen_Recording_20250117_063646_SOPT.DEBUG.mp4

To Reviewers

  • 우선 .. 2천 줄이 넘는 피알 죄송합니다 .. 😭😭 천천히.. 봐주세요...ㅎㅎ
  • 한 화면에 3개 상태를 (REGISTER, CHANGE, SEARCH) 관리해 줘야 되다 보니까 코드가 길어지고 복잡해진 것 같아요..
  • CertificationViewModel의 타이머 부분에서 타임아웃 시 , 화면 전환 시에도 Job을 취소해주는 로직 추가해줬어요!
  • 브렌치 머지는 feature/login-main으로 가게 해뒀습니다
  • 기본적으로 TODO 주석들과 서버통신 토스트는 수정될 예정입니다!

leeeyubin and others added 30 commits December 31, 2024 19:12
@leeeyubin leeeyubin self-assigned this Jan 16, 2025
@leeeyubin leeeyubin requested a review from a team as a code owner January 16, 2025 21:18
Copy link

height bot commented Jan 16, 2025

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants