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

[AN/USER] refactor: Hilt 적용(#453) #454

Merged
merged 9 commits into from
Sep 20, 2023
Merged

[AN/USER] refactor: Hilt 적용(#453) #454

merged 9 commits into from
Sep 20, 2023

Conversation

EmilyCh0
Copy link
Member

@EmilyCh0 EmilyCh0 commented Sep 19, 2023

📌 관련 이슈

✨ PR 세부 내용

  • Hilt 사용해서 수동 DI를 자동 DI로 바꾼다
  • 현재 전부 Singleton, SingletonComponent -> ViewModelComponent와 Singleton 아닌 것 분리

@EmilyCh0 EmilyCh0 added the AN 안드로이드에 관련된 작업 label Sep 19, 2023
@EmilyCh0 EmilyCh0 linked an issue Sep 19, 2023 that may be closed by this pull request
@EmilyCh0 EmilyCh0 added USER ⚙️ 리팩터링 리팩터링에 관련된 작업 labels Sep 19, 2023
Copy link
Collaborator

@re4rk re4rk left a comment

Choose a reason for hiding this comment

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

LGTM👍

.build()

@Provides
@Singleton
Copy link
Collaborator

Choose a reason for hiding this comment

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

반환 타입이 원시 타입이라 Qualifier를 미리 정의하는건 어떤가요??

Copy link
Member Author

Choose a reason for hiding this comment

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

BaseUrlQualifier 추가 완료!

Copy link
Member

@SeongHoonC SeongHoonC left a comment

Choose a reason for hiding this comment

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

드디어 힐트 적용이네요! 추가 커밋 고생 많으셨습니다!

@EmilyCh0 EmilyCh0 merged commit c1f8456 into dev Sep 20, 2023
1 check passed
@EmilyCh0 EmilyCh0 deleted the feat/#453 branch September 20, 2023 07:26
BGuga pushed a commit that referenced this pull request Oct 17, 2023
* chore: Hilt 의존성 추가

Co-authored-by: SeongHoonC <[email protected]>

* feat: 의존성 주입 모듈 추가

Co-authored-by: SeongHoonC <[email protected]>

* feat: 의존성 주입에 hilt 적용

Co-authored-by: SeongHoonC <[email protected]>

* refactor: 사용하지 않는 수동 di 코드 삭제

Co-authored-by: SeongHoonC <[email protected]>

* chore: ktlintcheck

Co-authored-by: SeongHoonC <[email protected]>

* refactor: 불필요한 어노테이션 제거

* refactor: 뷰모델 스코프 모듈 분리

* chore: 싱글톤 모듈 패키지 이동

* refactor: BaseUrlQualifier 추가

---------

Co-authored-by: SeongHoonC <[email protected]>
BGuga pushed a commit that referenced this pull request Oct 17, 2023
* chore: Hilt 의존성 추가

Co-authored-by: SeongHoonC <[email protected]>

* feat: 의존성 주입 모듈 추가

Co-authored-by: SeongHoonC <[email protected]>

* feat: 의존성 주입에 hilt 적용

Co-authored-by: SeongHoonC <[email protected]>

* refactor: 사용하지 않는 수동 di 코드 삭제

Co-authored-by: SeongHoonC <[email protected]>

* chore: ktlintcheck

Co-authored-by: SeongHoonC <[email protected]>

* refactor: 불필요한 어노테이션 제거

* refactor: 뷰모델 스코프 모듈 분리

* chore: 싱글톤 모듈 패키지 이동

* refactor: BaseUrlQualifier 추가

---------

Co-authored-by: SeongHoonC <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AN 안드로이드에 관련된 작업 USER ⚙️ 리팩터링 리팩터링에 관련된 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[AN] Hilt 적용
3 participants