[Feat] 자체 QA by kamja0510 · Pull Request #82 · DIGGING-PLAY/DPlay-Android

kamja0510 · 2026-01-18T08:33:26Z

Related issue 🛠

closed [Feat] UI 빠진 부분들 수정 #78

Work Description ✏️

base image 처리
탭 로우 아래 회색 박스 두기
splash 1초로 변경
등록된 곡들 클릭 시 상세 곡 페이지로 이동
navdisplay 애니메이션 없애기
마이페이지에서 삭제 로직 구현
웹뷰 구현
온보딩 페이지에 사진 추가

To Reviewers 📢

… 사용하는 것으로 변경

…역 bottomnavigation의 visible을 건들 수 있도록 구현

# Conflicts: # core/data/src/main/java/com/example/data/datasource/remote/UserRemoteDataSource.kt

Copilot

Pull request overview

이 PR은 자체 QA를 통해 발견된 여러 개선 사항들을 구현합니다. 주요 변경 사항으로는 프로필 이미지 처리 개선, 웹뷰 구현, 마이페이지 삭제 기능, 네비게이션 애니메이션 제거, UI 개선 등이 포함됩니다.

Changes:

프로필 이미지 처리를 Uri에서 String으로 변경하고 공통 컴포넌트(DPlayProfileImageArea) 추가
설정 및 온보딩 화면에 웹뷰 연결 구현 (약관, 공지사항, 문의 등)
마이페이지에서 등록한 곡 삭제 기능 및 상세 페이지 이동 구현
네비게이션 애니메이션 제거 및 스플래시 시간 단축 (2초→1초)
바텀 네비게이션 컨트롤러 추가 및 탭 로우 UI 개선
온보딩 페이지에 이미지 추가

Reviewed changes

Copilot reviewed 31 out of 35 changed files in this pull request and generated 11 comments.

Show a summary per file

File	Description
feature/splash/SplashViewModel.kt	스플래시 딜레이를 2초에서 1초로 단축
feature/setting/*	웹뷰 오픈 기능 추가 (공지사항, 문의, 약관 등)
feature/onboarding/*	프로필 이미지 String 처리, 약관 웹뷰, 알림 권한 업데이트, 네비게이션 애니메이션 제거, 이미지 추가
feature/mypage/*	삭제 기능 구현, 트랙 클릭 네비게이션, 바텀시트/다이얼로그 추가, 빈 화면 뷰 추가
feature/main/MainActivity.kt	바텀 네비게이션 컨트롤러 통합, 네비게이션 애니메이션 제거
feature/editprofile/EditProfileScreen.kt	DPlayProfileImageArea 컴포넌트로 리팩토링
feature/detail/DetailScreen.kt	기본 프로필 이미지 처리 추가
feature/comment/*	가이드 웹뷰 추가, 등록 후 홈으로 이동 변경
core/ui/controller/BottomNavigationController.kt	바텀 네비게이션 표시/숨김 컨트롤러 신규 추가
core/designsystem/component/DPlayProfileImageArea.kt	프로필 이미지 영역 공통 컴포넌트 신규 추가
core/designsystem/res/drawable/base_profile_image.png	기본 프로필 이미지 리소스 추가
core/domain/model/Writer.kt	profileImg를 nullable로 변경
core/data/*	PatchProfileRequest 추가, 멀티파트 요청 변경, 매퍼 수정
core/common/constant/Url.kt	URL 상수 객체 신규 추가
core/common/type/TermType.kt	약관 타입에 URL 추가

Copilot · 2026-01-18T08:39:25Z

feature/mypage/src/main/java/com/example/mypage/MyPageScreen.kt

+                ),
+            exit =
+                slideOutVertically(
+                    targetOffsetY = { 0 },


exit 애니메이션의 targetOffsetY가 0으로 설정되어 있어 바텀시트가 사라질 때 화면 상단으로 이동합니다. 이는 의도한 동작이 아닌 것으로 보입니다. slideOutVertically에서 targetOffsetY는 { it }로 설정하여 아래로 슬라이드되도록 해야 합니다.

Suggested change

targetOffsetY = { 0 },

targetOffsetY = { it }, // 바텀시트가 아래로 슬라이드되며 사라지도록 수정

Copilot · 2026-01-18T08:39:26Z

feature/main/src/main/java/com/example/main/MainActivity.kt

                                )
                            },
                        ) { padding ->
+                            val bottomPadding = if (navigator.shouldShowBottomSheet) padding.calculateBottomPadding() else 0.dp


bottomPadding 계산 로직이 중복되고 있습니다. navigator.shouldShowBottomSheet 조건이 BottomNavigationBar의 isVisible과 bottomPadding 계산에 모두 사용되는데, BottomNavigationBar가 보이지 않을 때도 padding이 적용될 수 있습니다. bottomNavigationController.bottomNavigationVisible도 함께 고려해야 합니다.

Copilot · 2026-01-18T08:39:26Z

core/designsystem/src/main/java/com/example/designsystem/component/DPlayProfileImageArea.kt

+            modifier =
+                Modifier
+                    .clip(CircleShape)
+                    .border(
+                        width = 1.dp,
+                        color = DPlayTheme.colors.gray200,
+                        shape = CircleShape,
+                    ),
+            contentScale = ContentScale.Crop,
+        )


Box에 적용된 modifier (size 포함)가 AsyncImage로 전달되지 않고 있습니다. AsyncImage에 matchParentSize() 또는 fillMaxSize()를 적용하지 않으면 이미지가 Box의 크기를 채우지 못합니다. AsyncImage의 modifier를 Modifier.matchParentSize()로 시작하도록 수정해야 합니다.

Copilot · 2026-01-18T08:39:26Z

feature/onboarding/src/main/java/com/example/onboarding/OnboardingViewModel.kt

+                            .updateNotificationEnabled(intent.isGranted)
+                            .onSuccess {
+                                setSideEffect(OnboardingContract.OnboardingSideEffect.NavigateToHome)
+                            }.onFailure {


알림 권한 설정 업데이트가 실패했을 때(onFailure) 처리 로직이 비어있습니다. 실패해도 홈 화면으로 이동하거나, 에러를 사용자에게 알려야 합니다. 현재는 실패 시 아무 동작도 하지 않아 사용자가 대기 상태에 갇힐 수 있습니다.

Suggested change

}.onFailure {

}.onFailure {

// 알림 설정 저장이 실패하더라도 사용자 흐름이 막히지 않도록 홈 화면으로 이동

setSideEffect(OnboardingContract.OnboardingSideEffect.NavigateToHome)

Copilot · 2026-01-18T08:39:27Z

core/common/src/main/java/com/example/common/constant/Url.kt

+    // 에러 뷰
+    const val ERROR = ""


ERROR 상수가 빈 문자열로 정의되어 있습니다. 빈 문자열을 URL로 사용하면 의도하지 않은 동작이 발생할 수 있습니다. null을 반환하거나 유효한 에러 페이지 URL을 제공하는 것이 좋습니다.

Suggested change

// 에러 뷰

const val ERROR = ""

// 에러 뷰 - 빈 문자열 대신 안전한 기본 에러 URL 사용

const val ERROR = "about:blank"

Copilot · 2026-01-18T08:39:27Z

core/data/src/main/java/com/example/data/service/UserService.kt

    suspend fun patchProfile(
        @Part profileImg: MultipartBody.Part?,
-        @Part("nickname") request: String?,
+        @Part("changeProfileRequest") request: PatchProfileRequest,


@part 어노테이션에서 @part("changeProfileRequest")로 복잡한 객체를 전달하고 있는데, Multipart 요청에서는 일반적으로 JSON 객체를 @part로 직접 전달할 수 없습니다. RequestBody로 변환하거나 각 필드를 개별 @part로 전달해야 합니다. 현재 구현은 서버 요청이 실패할 가능성이 높습니다.

Copilot · 2026-01-18T08:39:27Z

feature/mypage/src/main/java/com/example/mypage/MyPageContract.kt

        ) : MyPageIntent

        data class OnKebabIconClick(
            val musicId: Long,


OnKebabIconClick의 파라미터 이름이 musicId로 되어 있지만, 실제로는 postId를 받아야 합니다. OnScrappedTrackClick과 OnRegisteredTrackClick은 postId를 사용하므로 일관성을 위해 musicId를 postId로 변경해야 합니다.

Suggested change

val musicId: Long,

val postId: Long,

Copilot · 2026-01-18T08:39:28Z

feature/mypage/src/main/java/com/example/mypage/MyPageScreen.kt

+        Spacer(modifier = Modifier.height(184.dp))
+
+        Text(
+            text = "아직 등록한 곡이 없어요",


ScrappedMusicEmptyView의 텍스트가 RegisteredMusicEmptyView와 동일합니다. "아직 스크랩한 곡이 없어요" 또는 유사한 메시지로 변경되어야 합니다.

Suggested change

text = "아직 등록한 곡이 없어요",

text = "아직 스크랩한 곡이 없어요",

Copilot · 2026-01-18T08:39:28Z

feature/mypage/src/main/java/com/example/mypage/MyPageViewModel.kt

+                    updateState {
+                        copy(
+                            isDeleteBottomSheetVisible = true,
+                            selectedPostId = intent.musicId,


OnKebabIconClick intent의 파라미터 이름이 musicId로 되어 있지만 실제로는 postId를 전달받아야 합니다. 다른 intent들(OnRegisteredTrackClick, OnScrappedTrackClick)은 postId라는 명확한 이름을 사용하고 있으므로, 일관성을 위해 musicId를 postId로 변경해야 합니다.

Suggested change

selectedPostId = intent.musicId,

selectedPostId = intent.postId,

Copilot · 2026-01-18T08:39:28Z

feature/mypage/src/main/java/com/example/mypage/MyPageViewModel.kt

+                                    )
+                                }
+                            }.onFailure {
+                                Timber.d(t = it, message = "deletePost 실패")


삭제가 실패했을 때(onFailure) 사용자에게 에러 메시지를 표시하는 로직이 없습니다. 로그만 남기고 있어 사용자는 삭제가 실패했는지 알 수 없습니다. 에러 스낵바나 토스트를 통해 실패를 알려야 합니다.

Suggested change

Timber.d(t = it, message = "deletePost 실패")

// 삭제 실패 시 로그만 남기지 말고, UI에 에러를 알려 사용자 경험을 개선한다.

Timber.e(t = it, message = "deletePost 실패")

setSideEffect(

MyPageContract.MyPageSideEffect.ShowErrorToast(

message = "삭제에 실패했어요. 잠시 후 다시 시도해 주세요."

)

)

kamja0510 added 26 commits January 16, 2026 17:57

[feat] #78 URL 상수 정의

0da5765

[feat] #78 이용 약관 enum 타입에 Url 변수 추가

8b6ebe7

[feat] #78 약관 Row의 arrow 아이콘 클릭 시 노션 페이지 띄우도록 구현

c66a82c

[feat] #78 프로필 사진 기본 이미지 수정

24d5611

[feat] #78 온보딩 pager 1,2 페이지에 사진 추가

d446081

[refactor] #78 온보딩 프로필 사진 설정 화면에서 profileImagePath를 Uri객체에서 String 객체…

7e49627

… 사용하는 것으로 변경

[refactor] #78 프로필 사진 영역 컴포넌트화

7384ba4

[feat] #78 setting screen 웹뷰 띄우기 로직 구현

129b87a

[fix] #78 프로필 수정 api 에서 닉네임 수정 안되는 오류 수정

cd72590

[feat] #78 등록하기 클릭 후 홈으로 이동 구현

ff5c2fd

[feat] #78 커뮤니티 가이드 버튼 클릭 시 커뮤니티 가이드 노션 페이지로 이동 구현

c732e4f

[feat] #78 보관한 곡 클릭 시 곡 상세 페이지로 이동 구현

a2afd84

[feat] #78 MyPage Screen의 함수들 드릴링 하고 bottomNavigationController 만들어 전…

b1a21f9

…역 bottomnavigation의 visible을 건들 수 있도록 구현

[feat] #78 케밥 아이콘 클릭 시 바텀시트 올라오도록 구현

0bb63e0

Merge branch 'develop' into feat/#78-qa-1-ui

9e1be81

# Conflicts: # core/data/src/main/java/com/example/data/datasource/remote/UserRemoteDataSource.kt

[fix] #78 bottomsheet 디자인 수정

87f36f1

[feat] #78 등록한 곡 삭제 구현

6ec71d0

[feat] #78 navDisplay의 애니메이션 다 없에고 bottombar에 애니메이션 추가

a3f2b5f

[feat] #78 bottomBar에 애니메이션 되돌리기

7fa46e4

[feat] #78 등록된 곡 클릭 시 상세 곡 페이지로 이동

8a12512

[feat] #78 알림 권한 붙이기 아직 안됨

bdf8c38

[chore] #78 ktlint

57e98f6

[feat] #78 splash 시간 상수 1000L으로 변경

8433350

Merge branch 'develop' into feat/#78-qa-1-ui

24f5c6d

[feat] #78 baseimage 처리 구현

5c71b12

[feat] #78 탭 로우 아래에 회색 박스 두기

6c8df6c

kamja0510 requested a review from t1nm1ksun January 18, 2026 08:33

kamja0510 self-assigned this Jan 18, 2026

Copilot AI review requested due to automatic review settings January 18, 2026 08:33

kamja0510 added the 🥔재민 label Jan 18, 2026

kamja0510 added the 🪡Feat label Jan 18, 2026

Copilot started reviewing on behalf of kamja0510 January 18, 2026 08:33 View session

Copilot AI reviewed Jan 18, 2026

View reviewed changes

[chore] #78 ktlint

3a6789d

kamja0510 merged commit 02149b3 into develop Jan 21, 2026
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

[Feat] 자체 QA#82

[Feat] 자체 QA#82
kamja0510 merged 27 commits intodevelopfrom
feat/#78-qa-1-ui

kamja0510 commented Jan 18, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Jan 18, 2026

Uh oh!

Copilot AI Jan 18, 2026

Uh oh!

Copilot AI Jan 18, 2026

Uh oh!

Copilot AI Jan 18, 2026

Uh oh!

Copilot AI Jan 18, 2026

Uh oh!

Copilot AI Jan 18, 2026

Uh oh!

Copilot AI Jan 18, 2026

Uh oh!

Copilot AI Jan 18, 2026

Uh oh!

Copilot AI Jan 18, 2026

Uh oh!

Copilot AI Jan 18, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

	targetOffsetY = { 0 },
	targetOffsetY = { it }, // 바텀시트가 아래로 슬라이드되며 사라지도록 수정

	text = "아직 등록한 곡이 없어요",
	text = "아직 스크랩한 곡이 없어요",

	selectedPostId = intent.musicId,
	selectedPostId = intent.postId,

-                                Timber.d(t = it, message = "deletePost 실패")
+                                // 삭제 실패 시 로그만 남기지 말고, UI에 에러를 알려 사용자 경험을 개선한다.
+                                Timber.e(t = it, message = "deletePost 실패")
+                                setSideEffect(
+                                    MyPageContract.MyPageSideEffect.ShowErrorToast(
+                                        message = "삭제에 실패했어요. 잠시 후 다시 시도해 주세요."
+                                    )
+                                )

Comments

Conversation

kamja0510 commented Jan 18, 2026

Related issue 🛠

Work Description ✏️

To Reviewers 📢

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 18, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant