-
Notifications
You must be signed in to change notification settings - Fork 2
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
#291 [FEAT] domain 모듈 코틀린 라이브러리로 변경 #297
Conversation
@@ -5,7 +5,7 @@ import com.google.firebase.crashlytics.FirebaseCrashlytics | |||
import com.kakao.sdk.common.KakaoSdk | |||
import com.mument_android.BuildConfig | |||
import com.mument_android.data.BuildConfig.KAKAO_NATIVE_KEY | |||
import com.mument_android.domain.entity.musicdetail.musicdetaildata.Music | |||
import com.mument_android.domain.entity.musicdetail.Music |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
depth 줄임 MusicDetailEntity 없애서 dicrectory가 하나 덜 필요해졌으
@@ -20,7 +19,7 @@ class MoveMusicDetailNavigatorProviderImpl @Inject constructor(private val activ | |||
override fun intentMusicDetail(music: MusicInfoEntity) { | |||
with(activity as SearchActivity) { | |||
val intent = Intent(this, MainActivity::class.java).apply { | |||
putExtra("MUSIC_INFO_ENTITY", music) | |||
putExtra(MUSIC_INFO_ENTITY, music) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
상수처리
inline fun <reified T : Serializable> Bundle.getBundleSerializable(key: String): T? = when { | ||
SDK_INT >= TIRAMISU -> getSerializable(key, T::class.java) | ||
else -> @Suppress("DEPRECATION") getSerializable(key) as? T | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
버전 분기용 compat 추가
@@ -2,5 +2,5 @@ package com.mument_android.data.dto.record | |||
|
|||
data class MumentIsFirstDto( | |||
val isFirst : Boolean, | |||
val FirstAvaliable : Boolean | |||
val firstAvailable : Boolean |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네이밍 컨벤션 지킵시다!
return if (it.data != null) { | ||
setProfileMapper.map(it.data) | ||
} else null | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Multi Part 만드는 작업은 여기로 이관
addTestDependencies() | ||
addNetworkDependencies() | ||
addDaggerHiltDependencies() | ||
implementation(KotlinDependencies.annotation) | ||
implementation(KotlinDependencies.kotlinxSerialization) | ||
implementation(KotlinDependencies.coroutines) | ||
implementation(AndroidXDependencies.pagingCommon) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 안드로이드 의존성 아님, 테스트용 의존성
import android.widget.Toast | ||
import java.io.ByteArrayOutputStream | ||
|
||
private var toast: Toast? = null | ||
fun Context.shortToast(text: String) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 나중에 core-dependent 로 이동
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이렇게 보니깐 뭐 안 한것처럼 보이지만 겁나 오래 걸렸어요,,
고생하셨습니당...!!ㅎㅎ |
🎸 작업한 내용
🎶 PR Point
📸 스크린샷
💽 관련 이슈