🔀 :: (#553) Playlist 도메인을 v3로 업데이트합니다. #559

yongbeomkwak · 2024-06-02T15:19:15Z

💡 배경 및 개요

왁뮤 v3 api로 playlist 도메인을 최신화합니다.

Resolves: #553

📃 작업내용

현재까지 작업된 playlist 도메인 api로 최신화 합니다.
Entity 변화에 따른 비지니스로직을 최신화 합니다.

🙋‍♂️ 리뷰노트

구현 시에 고민이었던 점들 혹은 특정 부분에 대한 의도가 있었다면 PR 리뷰의 이해를 돕기 위해 서술해주세요!

또한 리뷰어에게 특정 부분에 대한 집중 혹은 코멘트 혹은 질문을 요청하는 경우에 작성하면 좋아요!

e.g. 작업을 끝내야할 시간이 얼마 없어 확장성보다는 동작을 위주로 만들었어요! 감안하고 리뷰해주세요!

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. XCConfig, 노션, README)
이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "XCConfig 값 추가되었어요")
작업한 코드가 정상적으로 동작하나요?
Merge 대상 브랜치가 올바른가요?
PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

github-actions · 2024-06-02T15:22:16Z

✅ Successful finished SwiftLint

baekteun · 2024-06-03T00:59:59Z

Projects/App/Sources/Application/AppComponent+Playlist.swift

        }
    }

-    var loadPlayListUseCase: any LoadPlayListUseCase {
+    var updateTitleAndPrivateeUseCase: any UpdateTitleAndPrivateeUseCase {


updateTitleAndPrivateeUseCase
Private"e" 네요

엇 수정했습니다.

baekteun · 2024-06-03T01:00:26Z

Projects/Domains/BaseDomain/Interface/Enum/ImageType.swift

+import Foundation
+
+public enum UploadImageType {
+    case `default`(data: String)


data보다는 imageName 이런식 어떤가요

반영하겠습니다 ~

baekteun · 2024-06-03T01:00:58Z

Projects/Domains/BaseDomain/Interface/Enum/ImageType.swift

+
+public enum UploadImageType {
+    case `default`(data: String)
+    case custom (data: Data)


파일이름같은거도 받는거 어떤가요

파일이름이요 ?? 서버에 보낼 때 쓰지 않는데 받을 필요가 있나요?

파일 이름 안보내면 서버에서 500띄워요

프레임워크단에서 터져요

흠 어떤식으로 하혐 좋을까요 ??
default는 저희가 제공해주는 기본 이미지들이고
custom이 갤러리에서 가져오는 것들인데

baekteun · 2024-06-03T01:02:00Z

Projects/Domains/PlayListDomain/Interface/UseCase/UploadPlaylistImageUseCase.swift

+import RxSwift
+
+public protocol UploadPlaylistImageUseCase {
+    func execute(key: String, model: UploadImageType) -> Single<BaseImageEntity>


그냥 String 리턴하는거 어떤가요? 감쌀필요가 있을까 라는 생각이 드네요.

엇 저는 통일성 때문에 쓰긴했는데.. 그게 나을까요 ??

baekteun · 2024-06-03T01:03:46Z

Projects/Domains/PlayListDomain/Sources/API/PlayListAPI.swift

+    case fetchPlaylistSongs(key: String) // 전체 재생 시 곡 데이터만 가져오기
+    case addSongIntoPlayList(key: String, songs: [String]) // 곡 추가
+    case updatePlaylist(key: String, songs: [String]) // 최종 저장
+    case removeSongs(key: String, songs: String)


songs는 [String]으로 받고 task 정의할때 .join(",") 하는거 어떤가요, 위에비해 일관성이 약간 떨어져보여서요
흠 애매하긴하네요

처리했습니다.

baekteun · 2024-06-03T01:05:59Z

Projects/Domains/PlayListDomain/Sources/API/PlayListAPI.swift

+                datas.append(MultipartFormData(provider: .data("default".data(using: .utf8)!), name: "type"))
+                datas.append(MultipartFormData(provider: .data(data.data(using: .utf8)!), name: "imageName"))
+
+            case let .custom(data: data):
+                datas.append(MultipartFormData(provider: .data("custom".data(using: .utf8)!), name: "type"))


! 들이 좀 많이 불편하긴하네요, 확정적이라해도. 코드상이 !가 여럿 있다는거만해도 !를 쓰는게 당연시되는 그런 효과가 생길거같아서

흠.. 그러면 ?? 연산자를 쓸까요?

baekteun · 2024-06-03T01:09:11Z

Projects/Domains/PlayListDomain/Sources/ResponseDTO/PlayListDetailResponseDTO.swift

    public let key: String?
    public let title: String
    public let songs: [SingleSongResponseDTO]?
-    public let image: SinglePlayListDetailResponseDTO.Image
-}
-
-public extension SinglePlayListDetailResponseDTO {
-    struct Image: Decodable {
-        public let round: Int?
-        public let square: Int?
-        public let name: String?
-        public let version: Int?
-    }
+    public let imageUrl: String?
+    public let `private`: Bool


key나 imageUrl이 옵셔널인 이유가 있을까요?

Projects/Features/BaseFeature/Interface/Enum/PurposeType.swift

baekteun · 2024-06-03T01:12:17Z

Projects/Features/PlaylistFeature/Sources/Components/PlayListDetailComponent.swift

@@ -27,7 +27,7 @@ public final class PlayListDetailComponent: Component<PlayListDetailDependency>,
                key: id,
                type: isCustom ? .custom : .wmRecommend,
                fetchPlayListDetailUseCase: dependency.fetchPlayListDetailUseCase,
-                editPlayListUseCase: dependency.editPlayListUseCase,
+                updatePlaylistUseCase: dependency.updatePlaylistUseCase,


흠.. 당장 반영되어야한다는건 아닌데 어디는 Play"L"ist 어디는 Play"l"ist 좀 여러식으로 혼용되고 있는거같네요, 한번 정하는 시간이 있는게 좋을거같아요

엇 .. 그러네요 전에 말씀드린 에셋 정리할 때 한번 해야겠네요 ..

…sic/wakmusic-iOS into 553-update-playlist-domain

baekteun · 2024-06-03T14:07:08Z

Projects/Domains/PlayListDomain/Sources/API/PlayListAPI.swift

+             let .removeSongs(
+                 key: key,
+                 _
+             ):


이 친구 뭐 변수인줄 알았는데 case별로 줄바꿈 한거네요? 가독성을 저해하는거같아서 이친구도 그냥 한 라인에 두는게 좋을거같아요

오잉 이거 제가 안했어요 ㅋㅋㅋ 포맷팅이 범인입니다 !

baekteun · 2024-06-03T14:08:50Z

Projects/Domains/PlayListDomain/Sources/API/PlayListAPI.swift

+                    provider: .data(data),
+                    name: "imageFile",
+                    fileName: "image.jpeg",
+                    mimeType: "image/jpeg, image/jpg, image/png"


mimeType 이거,, 이렇게 해도,, 되나요..? mimeType은 optional이라 안보내도 문제가 없었는데 형식은 nest에서 어떻게 처리되는지는 모르겠네요

흠 그러면 일단 빼볼께요 검색에서 참고했던게 다 명시를 해줘가지고

보낼거면 1개만,, 저런 스트링 셋트를 보낸적이 없어서 모르겠네요.

baekteun · 2024-06-03T14:15:04Z

Projects/Domains/PlayListDomain/Sources/API/PlayListAPI.swift

+                datas.append(MultipartFormData(
+                    provider: .data("default".data(using: .utf8)!),
+                    name: "type",
+                    fileName: "ㅅ"


여기는 그냥 지우셔도 될거같아요

제거했습니다.

baekteun · 2024-06-03T14:15:57Z

Projects/Domains/PlayListDomain/Interface/Entity/AddSongEntity.swift

    }

-    public let status: Int
    public let added_songs_length: Int


엄 snake_case네요,

반영했습니다.

yongbeomkwak added 27 commits May 31, 2024 21:37

♻️ :: /playlist/recommend/list , DTO, Entity 수정

434dc58

🧱 :: 포매팅

4c63ef1

⚡ :: 추천 플레이리스트 디테일 Entity, DTO 수정

dca61fa

🧱 :: 포맷팅

299930f

♻️ :: create api 리팩

2775a15

🧱 :: 포맷팅

5cf23dd

♻️ :: 플레이리스트 private 업데이트 및 곡 불러오기 api

2379a79

🧱 :: 포맷팅

b728556

♻️ :: 플레이스트에 곡 추가 구조 변경

ae89d51

🧱 :: 포맷팅

89c5c44

⚡ :: 플레이리스트 이름 변경 적용

1e4690a

♻️ :: 이름 변경 Api 리팩

373e92a

♻️ :: loadPlaylist 삭제

66572aa

🔥 :: 불필요한 리소스 제거

e655970

♻️ ::곡 삭제 api 리팩

e1dc906

⚡ :: patch 메소드 completable로

47dd1fc

🧱 :: 이미지 업로드 관련 작업 임시 커밋.

43dee0b

🧱 :: 포맷팅

3db0760

🧱 :: 임시 커밋

1102f5e

🧱 :: 포맷팅

246bd4e

♻️ :: assoticated value를 이용하여 정리

e46ba8d

🎨 :: 포맷팅

0b7b9f8

⚡ :: 이미지 DTO, Entity 생성

49b8c75

⚡ :: Usecase 구현

78aaed7

🎨 :: 포맷팅

ea1cee1

💉 :: uploadPlaylistImageUseCase 의존성 정의

8b345bf

🎨 :: 포맷팅

6baa3b4

yongbeomkwak requested review from kimdaehee0824, KangTaeHoon and youn9k as code owners June 2, 2024 15:19

yongbeomkwak requested a review from baekteun as a code owner June 2, 2024 15:19

Merge branch '549-renewal-before-search' into 553-update-playlist-domain

f39e81e

yongbeomkwak changed the title ~~🔀 :: (#553) 검색 전 화면을 v3 버전으로 리뉴얼합니다.~~ 🔀 :: (#553) Playlist 도메인을 v3로 업데이트합니다. Jun 2, 2024

Base automatically changed from 549-renewal-before-search to develop June 2, 2024 15:25

Merge branch 'develop' into 553-update-playlist-domain

c9394fe

baekteun reviewed Jun 3, 2024

View reviewed changes

yongbeomkwak added 6 commits June 3, 2024 13:13

💩 :: 오타 수정

7e64d6b

Merge branch '553-update-playlist-domain' of https://github.com/wakmu…

516ac85

…sic/wakmusic-iOS into 553-update-playlist-domain

💩 :: 오타 수정

80eb337

🎨 :: 코드 Formatting 적용

86b19be

🎨 :: fileName 명시

67d9275

🎨 :: 코드 Formatting 적용

af99d31

yongbeomkwak merged commit 87a0b65 into develop Jun 3, 2024
2 of 3 checks passed

yongbeomkwak deleted the 553-update-playlist-domain branch June 3, 2024 10:50

baekteun reviewed Jun 3, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🔀 :: (#553) Playlist 도메인을 v3로 업데이트합니다. #559

🔀 :: (#553) Playlist 도메인을 v3로 업데이트합니다. #559

yongbeomkwak commented Jun 2, 2024

github-actions bot commented Jun 2, 2024 •

edited

Loading

baekteun Jun 3, 2024

yongbeomkwak Jun 3, 2024

baekteun Jun 3, 2024

yongbeomkwak Jun 3, 2024

baekteun Jun 3, 2024

yongbeomkwak Jun 3, 2024

baekteun Jun 3, 2024

baekteun Jun 3, 2024

yongbeomkwak Jun 3, 2024

baekteun Jun 3, 2024

yongbeomkwak Jun 3, 2024

baekteun Jun 3, 2024

yongbeomkwak Jun 3, 2024

baekteun Jun 3, 2024

yongbeomkwak Jun 3, 2024

baekteun Jun 3, 2024

baekteun Jun 3, 2024

yongbeomkwak Jun 3, 2024

baekteun Jun 3, 2024 •

edited

Loading

yongbeomkwak Jun 3, 2024

baekteun Jun 3, 2024

yongbeomkwak Jun 3, 2024

KangTaeHoon Jun 3, 2024

baekteun Jun 3, 2024

yongbeomkwak Jun 3, 2024

baekteun Jun 3, 2024

yongbeomkwak Jun 3, 2024

🔀 :: (#553) Playlist 도메인을 v3로 업데이트합니다. #559

🔀 :: (#553) Playlist 도메인을 v3로 업데이트합니다. #559

Conversation

yongbeomkwak commented Jun 2, 2024

💡 배경 및 개요

📃 작업내용

🙋‍♂️ 리뷰노트

✅ PR 체크리스트

🎸 기타

github-actions bot commented Jun 2, 2024 • edited Loading

✅ Successful finished SwiftLint

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

baekteun Jun 3, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Jun 2, 2024 •

edited

Loading

baekteun Jun 3, 2024 •

edited

Loading