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

네트워크 매니저 리팩토링 #85

Merged
merged 15 commits into from
Sep 3, 2023

Conversation

enebin
Copy link
Member

@enebin enebin commented Aug 30, 2023

주요 변경사항

  1. Network를 shared 대신 @Dependency를 사용하게 변경
  2. Network API를 Main API - Sub API 구조로 변경
    // Main API
    public enum KeymeAPI {
        case myPage(MyPageAPI)
        case registerPushToken(RegisterPushTokenAPI)
        case auth(AuthorizationAPI)
        case registration(RegistrationAPI)
        case member(MemberAPI)
        case test(KeymeTestsAPI)
        case question(QuestionAPI)
    }
    
    // Sub API
    public enum MemberAPI {
        case fetch
    }
    ...
    • 원래는 API마다 Manager를 하나씩 만드려고 했으나 번거로움이 상상을 초월하는 관계로 이렇게 바꿈
    • Sub API 분류 기준은 Swagger 카테고리 기준(인데 실수로 인해 몇 가지 다른 게 있을 수 있음. 일단 원칙은 그러함...ㅎ)
  3. 이외 잔버그 수정, 푸시토큰 등록 로직 변경(등록 시 Access token이 필요하므로 appdelegate에서 하기 힘듦) 포함

@enebin enebin added the enhancement New feature or request label Aug 30, 2023
@enebin enebin requested review from k906506 and 0inn August 30, 2023 02:38
@enebin enebin self-assigned this Aug 30, 2023
@enebin enebin changed the title Refactor/network manager 네트워크 매니저 리팩토링 Aug 30, 2023
@enebin enebin mentioned this pull request Aug 29, 2023
Copy link
Contributor

@0inn 0inn left a comment

Choose a reason for hiding this comment

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

좋습니다 ! 고생하셨어요 👍🏻

Copy link
Contributor

@k906506 k906506 left a comment

Choose a reason for hiding this comment

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

일단 Swagger 기준으로 나눠놨다고 해서 그런거 같긴한데...
클래스 네이밍을 RegisterPushToken -> PushToken은 어떤가요? (Register 외에도 Remove나 Edit도 발생할 수 있으니...)
다른 API 클래스들은 명사형으로 되어 있어서 코멘트 남깁니다!

  • 리펙토링 완전 고생 많으셨습니다 😎

@enebin
Copy link
Member Author

enebin commented Sep 3, 2023

일단 Swagger 기준으로 나눠놨다고 해서 그런거 같긴한데... 클래스 네이밍을 RegisterPushToken -> PushToken은 어떤가요? (Register 외에도 Remove나 Edit도 발생할 수 있으니...) 다른 API 클래스들은 명사형으로 되어 있어서 코멘트 남깁니다!

  • 리펙토링 완전 고생 많으셨습니다 😎

변경했어용. 머지할게요~

@enebin enebin merged commit ee8d21f into refactor/code-cleansing(#75) Sep 3, 2023
1 check passed
@enebin enebin deleted the refactor/network-manager branch September 3, 2023 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants