Merge pull request #166 from YAPP-Github/fix/#161-archive-result

[fix] #161 ArchiveResult 화면별 Result 타입 분리

Author: ikseong00

app/src/main/java/com/neki/android/app/main/MainScreen.kt

@@ -37,7 +37,7 @@ import androidx.core.net.toUri
 import kotlinx.collections.immutable.ImmutableList
 import kotlinx.collections.immutable.persistentListOf
 import com.neki.android.feature.archive.api.ArchiveNavKey
-import com.neki.android.feature.archive.api.ArchiveResult
+import com.neki.android.feature.archive.api.PhotoUploadedResult
 import com.neki.android.feature.map.api.MapNavKey
 import com.neki.android.feature.mypage.api.MyPageNavKey
 import com.neki.android.feature.photo_upload.api.PhotoUploadNavKey
@@ -99,7 +99,7 @@ fun MainRoute(
             is MainSideEffect.NavigateToUploadAlbumWithGallery -> navigateToUploadAlbumWithGallery(sideEffect.uriStrings)
             is MainSideEffect.NavigateToUploadAlbumWithQRScan -> navigateToUploadAlbumWithQRScan(sideEffect.imageUrl)
             is MainSideEffect.ShowToast -> nekiToast.showToast(sideEffect.message)
-            MainSideEffect.RefreshArchive -> resultBus.sendResult<ArchiveResult>(result = ArchiveResult.PhotoUploaded)
+            MainSideEffect.RefreshArchive -> resultBus.sendResult(result = PhotoUploadedResult, allowDuplicate = false)
         }
     }
 

core/data-api/src/main/java/com/neki/android/core/dataapi/repository/PhotoRepository.kt

@@ -25,6 +25,8 @@ interface PhotoRepository {
 
     suspend fun updateFavorite(photoId: Long, favorite: Boolean): Result<Unit>
 
+    suspend fun updateMemo(photoId: Long, memo: String): Result<Unit>
+
     suspend fun getPhotosPage(
         folderId: Long? = null,
         page: Int = 0,

core/data/src/main/java/com/neki/android/core/data/remote/api/PhotoService.kt

@@ -3,6 +3,7 @@ package com.neki.android.core.data.remote.api
 import com.neki.android.core.data.remote.model.request.DeletePhotoRequest
 import com.neki.android.core.data.remote.model.request.RegisterPhotoRequest
 import com.neki.android.core.data.remote.model.request.UpdateFavoriteRequest
+import com.neki.android.core.data.remote.model.request.UpdateMemoRequest
 import com.neki.android.core.data.remote.model.response.BasicNullableResponse
 import com.neki.android.core.data.remote.model.response.BasicResponse
 import com.neki.android.core.data.remote.model.response.FavoriteSummaryResponse
@@ -15,6 +16,7 @@ import io.ktor.client.request.get
 import io.ktor.client.request.parameter
 import io.ktor.client.request.patch
 import io.ktor.client.request.post
+import io.ktor.client.request.put
 import io.ktor.client.request.setBody
 import javax.inject.Inject
 
@@ -53,6 +55,13 @@ class PhotoService @Inject constructor(
         }.body()
     }
 
+    // 메모 수정
+    suspend fun updateMemo(photoId: Long, memo: String): BasicNullableResponse<Unit> {
+        return client.put("/api/photos/$photoId") {
+            setBody(UpdateMemoRequest(memo))
+        }.body()
+    }
+
     // 즐겨찾는 앨범 조회
     suspend fun getFavoritePhotos(
         page: Int = 0,

core/data/src/main/java/com/neki/android/core/data/remote/model/request/UpdateMemoRequest.kt

@@ -0,0 +1,9 @@
+package com.neki.android.core.data.remote.model.request
+
+import kotlinx.serialization.SerialName
+import kotlinx.serialization.Serializable
+
+@Serializable
+data class UpdateMemoRequest(
+    @SerialName("memo") val memo: String,
+)

core/data/src/main/java/com/neki/android/core/data/remote/model/response/PhotoResponse.kt

@@ -20,6 +20,7 @@ data class PhotoResponse(
         @SerialName("width") val width: Int? = null,
         @SerialName("height") val height: Int? = null,
         @SerialName("createdAt") val createdAt: String,
+        @SerialName("memo") val memo: String? = null,
     ) {
         internal fun toModel() = Photo(
             id = photoId,
@@ -29,6 +30,7 @@ data class PhotoResponse(
             height = height,
             date = createdAt.toFormattedDate(),
             contentType = ContentType.fromString(contentType),
+            memo = memo.orEmpty(),
         )
     }
 

core/data/src/main/java/com/neki/android/core/data/repository/impl/PhotoRepositoryImpl.kt

@@ -65,6 +65,10 @@ class PhotoRepositoryImpl @Inject constructor(
         photoService.updateFavorite(photoId, favorite)
     }
 
+    override suspend fun updateMemo(photoId: Long, memo: String): Result<Unit> = runSuspendCatching {
+        photoService.updateMemo(photoId, memo)
+    }
+
     override suspend fun getPhotosPage(
         folderId: Long?,
         page: Int,

core/designsystem/src/main/res/drawable/icon_memo.xml

@@ -0,0 +1,23 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    android:width="28dp"
+    android:height="28dp"
+    android:viewportWidth="28"
+    android:viewportHeight="28">
+  <path
+      android:pathData="M11.698,22.845H4.45C3.649,22.845 3,22.196 3,21.396V11.849C3,11.464 3.153,11.095 3.425,10.824L9.824,4.425C10.095,4.153 10.464,4 10.849,4H20.396C21.196,4 21.845,4.649 21.845,5.45V11.248"
+      android:strokeLineJoin="round"
+      android:strokeWidth="1.93285"
+      android:fillColor="#00000000"
+      android:strokeColor="#4F525F"
+      android:strokeLineCap="round"/>
+  <path
+      android:pathData="M3,11.731H8.315C9.116,11.731 9.765,11.082 9.765,10.282V4"
+      android:strokeWidth="1.93285"
+      android:fillColor="#00000000"
+      android:strokeColor="#4F525F"/>
+  <path
+      android:pathData="M20.699,14.347L15.035,20.012C14.91,20.137 14.822,20.294 14.78,20.465L14.37,22.141C14.196,22.85 14.84,23.487 15.547,23.308L17.204,22.886C17.373,22.843 17.527,22.755 17.649,22.632L23.317,16.965C23.694,16.588 23.694,15.976 23.317,15.598L22.066,14.347C21.689,13.97 21.077,13.97 20.699,14.347Z"
+      android:strokeWidth="1.93285"
+      android:fillColor="#00000000"
+      android:strokeColor="#4F525F"/>
+</vector>

core/model/src/main/java/com/neki/android/core/model/Photo.kt

@@ -13,4 +13,5 @@ data class Photo(
     val width: Int? = null,
     val height: Int? = null,
     val contentType: ContentType = ContentType.JPEG,
+    val memo: String = "",
 )

feature/archive/api/src/main/kotlin/com/neki/android/feature/archive/api/ArchiveResult.kt

@@ -1,11 +1,13 @@
 package com.neki.android.feature.archive.api
 
-sealed interface ArchiveResult {
-    data class PhotoDeleted(val photoId: List<Long>) : ArchiveResult {
-        constructor(photoId: Long) : this(listOf(photoId))
-    }
+sealed interface ArchiveResult
 
-    data class FavoriteChanged(val photoId: Long, val isFavorite: Boolean) : ArchiveResult
+data object PhotoDetailResult : ArchiveResult
 
-    data object PhotoUploaded : ArchiveResult
-}
+data object AlbumDetailResult : ArchiveResult
+
+data object AllPhotoResult : ArchiveResult
+
+data object AllAlbumResult : ArchiveResult
+
+data object PhotoUploadedResult : ArchiveResult

feature/archive/impl/build.gradle.kts

@@ -12,4 +12,5 @@ dependencies {
 
     implementation(libs.androidx.activity.compose)
     implementation(libs.androidx.paging.compose)
+    implementation(libs.zoomable)
 }