Skip to content

[Feature] #36 네트워크 의존성 주입(DI) 구조로 리팩터링#37

Merged
kdn0325 merged 12 commits intodevfrom
feature/#36-network-di-refactor
Jun 12, 2025
Merged

[Feature] #36 네트워크 의존성 주입(DI) 구조로 리팩터링#37
kdn0325 merged 12 commits intodevfrom
feature/#36-network-di-refactor

Conversation

@kdn0325
Copy link
Copy Markdown
Collaborator

@kdn0325 kdn0325 commented Jun 11, 2025

  • APIClient 및 PixabayVideoService 싱글톤 의존 제거
  • 의존성 주입 기반으로 설계 변경
  • URLRequest 구성 분리 (Endpoint)
  • 테스트 가능하고 확장성 높은 구조로 개선

@korbit-ai
Copy link
Copy Markdown

korbit-ai Bot commented Jun 11, 2025

You've used up your 5 PR reviews for this month under the Korbit Starter Plan. You'll get 5 more reviews on June 22nd, 2025 or you can upgrade to Pro for unlimited PR reviews and enhanced features in your Korbit Console.

Copy link
Copy Markdown
Collaborator

@vinyl-nyl vinyl-nyl 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
Copy Markdown
Collaborator

@jihyeonjjang jihyeonjjang 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
Copy Markdown
Collaborator

@giseungNoh giseungNoh left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@kdn0325 kdn0325 changed the title [Feature] #36 network di refactor [Feature] #36 네트워크 의존성 주입(DI) 구조로 리팩터링 Jun 12, 2025
Copy link
Copy Markdown
Owner

@TeddKo TeddKo left a comment

Choose a reason for hiding this comment

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

👏

Comment thread PickaView/Data/Network/APIClient.swift
@kdn0325 kdn0325 merged commit ef8e8a6 into dev Jun 12, 2025
@kdn0325 kdn0325 deleted the feature/#36-network-di-refactor branch June 13, 2025 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants