Skip to content

[#30] 로그인 기능 추가 #41

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
Open
ethan-223 wants to merge 20 commits into
base: develop
Choose a base branch
from
Open

[#30] 로그인 기능 추가 #41

ethan-223 wants to merge 20 commits into from

Conversation

@ethan-223
Copy link
Collaborator

@ethan-223 ethan-223 commented Aug 4, 2022

내용

  • �로그인 및 회원가입 기능을 구현합니다.

작업 사항

@ethan-223 ethan-223 added the feature 새로운 기능 label Aug 4, 2022
@ethan-223 ethan-223 self-assigned this Aug 4, 2022
@yuni-ju
Copy link
Member

yuni-ju commented Aug 21, 2022

구글 로그인 잘 되는 것을 확인했습니다!! 👍
그런데 개인 정보 입력 후 다음으로 넘어갈 시 저만 100번 에러케이스가 뜨는 걸까요..!?
Error 문서를 확인해보니 Client 요청값과 관련된 에러라고해서 남겨봅니다

@yuni-ju
Copy link
Member

yuni-ju commented Aug 21, 2022

닉네임 설정 EditText에서 닉네임을 입력하고 지울 시 커서가 가장 왼쪽으로 고정되는 현상이 발생하는 것 같습니다.

image

@heechokim
Copy link
Collaborator

heechokim commented Aug 29, 2022

@Gemij @winter223
만약 윤겸님이 발견해주신 이슈 해결이 오래 걸린다면 별도 PR을 만들어서 해결하는 것도 좋을 것 같아요ㅎㅎ
스프린트 기간이 짧아져서 리뷰 기간을 줄여야 할 것 같아서요..ㅠㅡㅠ

@ethan-223
Copy link
Collaborator Author

ethan-223 commented Sep 4, 2022

@Gemij
서버 쪽에서 뭔가 수정이 있었던 것 같더라구요. api 문서상 회원가입이 사라졌고 로그인도 url 이 바뀌어있네요.
문의 드려놓았고 답변 오면 맞춰서 대응하겠읍니다.
커서가 왼쪽으로 옮겨가는 이슈는 제가 CharSequence 를 equals 로 비교하고 있었네요. 수정했습니다 ㅎ

Base automatically changed from feature/response_body_converter to develop September 4, 2022 22:58
@ethan-223
[#30] 로그인 기능 추가
a4070d4 
- UI 작업
@ethan-223
[#30] 로그인 기능 추가
ad6d58b 
- Google OneTabSiginin 작업
@ethan-223
[#30] 로그인 기능 추가
8e6dece 
- Firebase SiginIn 작업
@ethan-223
[#30] 로그인 기능 추가
0d8b260 
- ProfileEditActivity 추가
- 이름과 닉네임 입력이 가능하고, ViewModel 의 UiModel 을 뷰에 반영시키도록 함
@ethan-223
[#30] 로그인 기능 추가
c11241d 
- Firebase 로그인 기능 제거 (기획 확인 됨)
- 패키지 정리
@ethan-223
[#30] 로그인 기능 추가
71d3f4a 
- 이미지 선택 다이얼로그 추가
@ethan-223
[#30] 로그인 기능 추가
b05ecc9 
- 신규 유저일 경우, IdToken, ProviderType 을 ProfileEditActivity로 전달
- ProfileEditActivity 에서 다음 버튼을 눌렀을 때 IdToken 과 ProviderType, 입력한 프로필 정보를 바탕으로 회원가입 진행할 수 있도록 밑작업
@ethan-223
[#30] 로그인 기능 추가
a0f8c76 
- 갤러리에서 이미지 선택 기능 추가
@ethan-223
[#30] 로그인 기능 추가
afdd4c6 
- 성별 선택 기능 추가
- 바뀐 디자인 가이드 소폭 적용
@ethan-223
[#30] 로그인 기능 추가
9d57c4d 
- 패키지 정리
- 네이밍 수정
- Dispatcher hilt 로 주입
@ethan-223
[#30] 로그인 기능 추가
e7a17a1 
- 스타일 정리
@ethan-223
[#30] 로그인 기능 추가
2c939ad 
- 프로필 필드 유효성 검사를 뷰모델에서 실행
- 모두 유효한 경우에만 버튼 활성화
- 필드 V 아이콘 제거
- RoundButton 비활성화 시 텍스트 색상 화이트로 수정
@ethan-223
[#30] 로그인 기능 추가 - 회원가입 요청
8fa4b15
@ethan-223
[#30] 로그인 기능 추가 - 로그인 요청
ec976a1
@ethan-223
[#30] 로그인 기능 추가 - 닉네임 필드 제거
286f0a8
@ethan-223
[#30] 로그인 기능 추가 - converter 적용
98fc308
@ethan-223
[#30] 로그인 기능 추가 - 불필요한 코드 제거
5fadb5a
@ethan-223
[#30] 로그인 기능 추가 - 불필요한 유틸 의존 제거
48a5021
@ethan-223 ethan-223 force-pushed the feature/login branch 2 times, most recently from 8bd1445 to b39916d Compare September 4, 2022 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yuni-ju yuni-ju yuni-ju approved these changes

@heechokim heechokim heechokim approved these changes

Assignees

@ethan-223 ethan-223

Labels

feature 새로운 기능

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ethan-223 @yuni-ju @heechokim