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

#17 - Refactor Firebase Processing Logic #18

Open

godpp wants to merge 1 commit into develop from feature/Remote

Member

godpp commented Aug 12, 2020 •

edited

Loading

Background

현재 계층화 되지 않은 Remote 모듈에 대해 계층화를 통해 Success, Error Handling에 대해 용이하게 하기 위함

추후 Framework로 Remote 모듈을 개발하기 위한 사전 작업

Task 1 - Firebase 모듈화 작업

FirebaseClient
- Firebase Instance를 통해 직접 Remote하는 wrapping class
FirebaseResult
- Firebase Remote 결과를 처리하는 Generic Enum
FirebaseDecoder
- Model Decoder

Task 2 - Firebase Database Path

static property로 관리

Task 3 - Firebase Login Flow 개선

Account Type과 로그인 결과 로직 분리


          <REFACTOR> Refactor Firebase Processing Logic

f11f513

godpp added refactor needTesting labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needTesting refactor