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

[TNT-140] DataStore 구축 #32

Merged
merged 8 commits into from
Jan 25, 2025
Merged

[TNT-140] DataStore 구축 #32

merged 8 commits into from
Jan 25, 2025

Conversation

hoyahozz
Copy link
Member

@hoyahozz hoyahozz commented Jan 24, 2025

📝 작업 내용

  • Closes [TNT-140] DataStore 구축 #21

  • DataStore 를 구축하였습니다.

    • data:storage 모듈 내 위치합니다.
    • storage 라고 명명한 이유는, datastore, preferences 와 같이 내부 저장소 라이브러리의 이름을 명시하고 싶지 않았기 때문입니다.
    • 정말 먼 훗날엔 dataStore 보다 더 좋은 라이브러리가 생길 수도 있는데, 그렇게 되면 모듈명 자체를 변경해야 합니다. (물론 큰 여파는 없을 것 같지만요..)

  • :data 모듈 계층을 변경하였습니다.
    • as-is -> only :data
    • to-be -> :data:repository <- :data:network, :data:storage
    • repository 는 여러 datasource 들을 취합하여 애플리케이션에 적합한 형태로 데이터를 매핑하는 역할을 맡고 있습니다.
    • 이에 따라 datasource 들을 가지고 있는 모듈과 repository 모듈을 분리하여 역할을 명시하였습니다.

📸 실행 화면

  • NONE

🙆🏻 리뷰 요청 사항

  • 혹시 궁금하신 점 있으시면 편하게 코멘트 달아주세여~

👀 레퍼런스

@hoyahozz hoyahozz added ✨ Feat 기능 구현 🌲 정호 김씨 집안 큰행님 김정호 labels Jan 24, 2025
@github-actions github-actions bot requested a review from SeonJeongk January 24, 2025 13:16
Copy link
Contributor

@SeonJeongk SeonJeongk left a comment

Choose a reason for hiding this comment

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

LGTM👍
코맨 하나만 확인해주세용

Comment on lines +12 to +15
@Singleton
class SessionDataSource @Inject constructor(
private val sessionPreferences: DataStore<Preferences>,
) {
Copy link
Contributor

Choose a reason for hiding this comment

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

로컬 저장 용도로 DataStore 사용할 때 DataSource를 나누는 기준은 어떻게 될까요?!

예를들어 SessionDataStore가 세션 관련 데이터를 관리하고 있는 것처럼 데이터의 용도에 따라 나누면 될까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

넵넵 맞습니다~!!

@hoyahozz hoyahozz merged commit 7a2f95c into develop Jan 25, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 기능 구현 🌲 정호 김씨 집안 큰행님 김정호
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TNT-140] DataStore 구축
2 participants