[Feat] 1차 QA 기능 추가 by kamja0510 · Pull Request #90 · DIGGING-PLAY/DPlay-Android

kamja0510 · 2026-01-28T00:16:11Z

Related issue 🛠

closed [Feat] 1차 QA 기능 추가 #89

Work Description ✏️

타 유저 프로필 볼 수 있도록 구현
매일 질문이 도착했다는 알림 구현
마이페이지의 scrapped 데이터들이 다른 화면을 갔다 돌아올 시 반영되도록 구현
앱 아이콘 추가
알림 권한이 시스템 설정과 연동되도록 구현
홈화면 large cover 블러 처리

Screenshot 📷

To Reviewers 📢

QA한번 시원하게 부탁드립니다~

Copilot

Pull request overview

이 PR은 1차 QA 과정에서 발견된 기능들을 구현합니다. 주요 변경사항으로는 타 유저 프로필 조회, 매일 오전 9시 질문 도착 알림, 마이페이지 데이터 새로고침, 앱 아이콘 추가, 알림 권한 시스템 연동, 홈화면 커버 블러 효과 등이 있습니다.

Changes:

새로운 feature:otherprofile 모듈 추가로 타 유저 프로필 조회 기능 구현
WorkManager 기반 매일 알림 스케줄링 및 알림 권한 시스템 설정 연동
Refresh Trigger 패턴을 통한 화면 간 데이터 동기화 개선
Badges 모델을 BADGE enum으로 리팩토링하여 타입 안정성 향상
MyPage를 data class로 변경하여 초기 탭 지정 가능

Reviewed changes

Copilot reviewed 56 out of 72 changed files in this pull request and generated 14 comments.

Show a summary per file

File	Description
settings.gradle.kts	otherprofile 모듈 추가
gradle/libs.versions.toml	WorkManager 의존성 추가
feature/setting/*	알림 권한을 시스템 설정으로 연동하도록 변경
feature/otherprofile/*	타 유저 프로필 조회 기능 신규 구현
feature/mypage/*	데이터 새로고침 trigger 통합 및 초기 탭 설정 기능 추가
feature/home/*	새로고침 trigger, 잠긴 커버 모달, 블러 효과 구현
feature/detail/*	삭제 확인 모달 추가, 날짜 표시 개선, BADGE 통합
feature/comment/*	Home 새로고침 trigger 연동
core/navigation/*	MyPage data class 변경, OtherProfile 라우트 추가
core/domain/*	UseCase 리팩토링, BADGE enum 도입, UserRelation 및 CheckUserRelationUseCase 추가
core/data/*	BADGE 모델 변경, PostDetail 날짜 포맷팅 추가, User 조회 API 추가
core/designsystem/*	DPlayLargeCover 블러 효과, 북마크 버튼 분리, 리포트 바텀시트 단일 선택으로 변경
core/common/*	HomeRefreshTrigger, RegisteredTrackRefreshTrigger, ScrappedTrackRefreshTrigger 추가
app/*	앱 아이콘 추가, DailyQuestionWorker 구현, 알림 스케줄링

Copilot · 2026-01-28T12:52:37Z

core/domain/src/main/java/com/example/domain/usecase/CheckUserRelationUseCase.kt

+    }
+
+    companion object{
+        const val ADMIN_ID = 1L


ADMIN_ID가 하드코딩되어 있습니다. 설정이나 BuildConfig를 통해 관리하는 것이 더 유연합니다.

Copilot · 2026-01-28T12:52:38Z

feature/detail/src/main/java/com/example/detail/DetailViewModel.kt

                    }.onFailure { e ->
-                        changeBottomSheetVisible(visible = false)
                        Timber.e(e)


포스트 삭제 실패 시 빈 onFailure 블록만 있습니다. 사용자에게 삭제 실패를 알리거나 에러를 로깅해야 합니다.

Copilot · 2026-01-28T12:52:38Z

feature/otherprofile/src/main/java/com/example/otherprofile/OtherProfileScreen.kt

+                } else {
+                }


빈 else 블록이 남아있습니다. 불필요한 코드는 제거하는 것이 좋습니다.

Copilot · 2026-01-28T12:52:38Z

feature/otherprofile/src/main/java/com/example/otherprofile/OtherProfileScreen.kt

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


하드코딩된 문자열입니다. strings.xml로 리소스화하는 것이 좋습니다.

Copilot · 2026-01-28T12:52:38Z

feature/otherprofile/src/main/java/com/example/otherprofile/OtherProfileScreen.kt

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


하드코딩된 문자열입니다. strings.xml로 리소스화하는 것이 좋습니다.

Copilot · 2026-01-28T12:52:40Z

core/data/src/main/java/com/example/data/repository/UserRepositoryImpl.kt

+                // 추후 Writer와 통합
+                User(
+                    id = userData.userId,
+                    nickname = userData.nickname,
+                    profileImagePath = userData.profileImg,
+                )


주석 "추후 Writer와 통합"이 있지만, Writer 타입과 User 타입이 중복되는 것으로 보입니다. 이는 기술 부채가 될 수 있으니 이슈로 트래킹하거나 TODO로 명확히 표시하는 것이 좋습니다.

Copilot · 2026-01-28T12:52:40Z

feature/otherprofile/src/main/res/values/strings.xml

@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="utf-8"?>
+<resources>
+    <string name="other_profile_screen_title">마이페이지</string>


타 유저의 프로필 화면인데 screen_title이 "마이페이지"로 되어 있습니다. 다른 사용자의 프로필이므로 "프로필" 또는 사용자 닉네임을 표시하는 것이 더 적절합니다.

Copilot · 2026-01-28T12:52:40Z

app/src/main/java/com/dplay/DPlayApplication.kt

+    private fun scheduleDailyNotification() {
+        val currentDate = Calendar.getInstance()
+        val dueDate = Calendar.getInstance()
+
+        dueDate.set(Calendar.HOUR_OF_DAY, 9)
+        dueDate.set(Calendar.MINUTE, 0)
+        dueDate.set(Calendar.SECOND, 0)
+
+        if (dueDate.before(currentDate)) {
+            dueDate.add(Calendar.HOUR_OF_DAY, 24)
+        }
+
+        val timeDiff = dueDate.timeInMillis - currentDate.timeInMillis
+
+        val dailyWorkRequest =
+            PeriodicWorkRequestBuilder<DailyQuestionWorker>(24, TimeUnit.HOURS)
+                .setInitialDelay(timeDiff, TimeUnit.MILLISECONDS)
+                .addTag(DailyQuestionWorker.TAG)
+                .build()
+
+        WorkManager
+            .getInstance(this)
+            .enqueueUniquePeriodicWork(
+                DailyQuestionWorker.WORK_NAME,
+                ExistingPeriodicWorkPolicy.KEEP,
+                dailyWorkRequest,
+            )


WorkManager는 정확한 시간 보장이 어렵습니다. 정확히 오전 9시에 알림을 보내야 한다면 AlarmManager 사용을 고려해야 합니다. WorkManager는 배터리 최적화를 위해 실행 시간을 조정할 수 있어 최대 15분 정도 지연될 수 있습니다.

Copilot · 2026-01-28T12:52:41Z

app/src/main/java/com/dplay/worker/DailyQuestionWorker.kt

+    override fun doWork(): Result {
+        showNotification()
+        return Result.success()
+    }


알림 권한이 없을 때 알림이 표시되지 않습니다. 권한 체크를 추가하거나, 최소한 권한이 없을 때는 Worker 실행을 건너뛰는 로직이 필요합니다.

Copilot · 2026-01-28T12:52:41Z

feature/setting/src/main/java/com/example/setting/SettingScreen.kt

+                if (event == Lifecycle.Event.ON_RESUME) {
+                    val isGranted = NotificationManagerCompat.from(context).areNotificationsEnabled()
+                    viewModel.handleIntent(SettingContract.SettingIntent.UpdateNotificationPermission(isGranted))
+                }


시스템 설정에서 알림 권한 변경 시 앱으로 돌아올 때마다 권한 상태를 확인하는 것은 좋은 접근이지만, 화면이 표시될 때마다(ON_RESUME) 체크하는 것은 과도할 수 있습니다. ON_START 이벤트를 사용하거나, 설정 화면에서 돌아올 때만 체크하는 것을 고려해보세요.

t1nm1ksun and others added 28 commits January 25, 2026 16:47

[refactor] #87 Apply BADGE enum class

fca9f48

[refactor] #87 date 서버응답 반영

e8561fb

[refactor] #87 상세화면 badge 전달

32a4b6a

[refactor] #87 홈 북마크 위치 고정

713a9a5

[refactor] #87 홈 뷰모델 init으로 초기 데이터 로드

9ae0516

[refactor] #87 오늘 노래가 3개여도 잠금 ui 표시

acfaca0

[refactor] #87 잠금 앨범 모달 추가

10b9e31

[feat] #87 상세화면 글 삭제 확인 모달 추가

48fe881

[feat] #87 글 등록/삭제 후 홈 데이터 새로고침

95dc714

[feat] #87 노래제목, 가수 한줄제한추가

1ad2e0e

[feat] #87 북마크 스낵바 네비게이션 연결

50238c1

[feat] #87 신고하기 바텀시 단일선택, 버튼 활성화 로직 추가

2f93cc6

[feat] #87 마이페이지 탭 유지 로직 추가

65d9593

[feat] #87 홈 새로고침시 첫번쨰 노래로 이동

fc1d649

ktlint Format

47b1bc5

[feat] #87 과거 추천 기록 노래 리스트 아이템 미트볼 버튼 삭제

5228963

[feat] #89 오늘의 질문 알림 구현

c633783

[feat] #89 타유저 프로필에도 사용하기 위한 보관함, 등록한 곡 가져오기 UseCase 로직 변경

1b2db85

[feat] #89 타유저 정보 가져오는 repository 구현

f340b2e

[feat] #89 타유저 프로필 모듈 생성 및 NavKey 생성

a39018a

[feat] #89 타유저 프로필 구현 완료

572d2a3

[feat] #89 userId로 에디터, 타유저, 자신 구분 하는 usecase 구현

d979ea5

[feat] #89 타유저 프로필 클릭 시 이동 UI에 연동

1fa7ffd

[feat] #89 mypage -> detail 이동 후 스크랩 한 뒤 뒤로 돌아왔을 때 변경사항 적용되도록 구현

56cb4a2

[feat] #89 앱 아이콘 설정

a6b4c9e

[feat] #89 알림 권한 시스템 설정과 연동되도록 구현

bfdc315

[chore] #89 ktlint

538b2d7

[feat] #89 등록된 곡 -> 상세 화면 이동 후 삭제한 후에 돌아왔을 때 데이터 동기화 되도록 구현

a0289bb

kamja0510 self-assigned this Jan 28, 2026

kamja0510 added the 🥔재민 label Jan 28, 2026

kamja0510 added the 🪡Feat label Jan 28, 2026

kamja0510 added 3 commits January 28, 2026 19:54

[feat] #89 home 화면 LargeCover 블러처리

5853edf

[feat] #89 guidelineButton end padding 12.dp로 수정

1c9c835

[chore] #89 ktlint

1db9215

kamja0510 requested review from Copilot and t1nm1ksun January 28, 2026 12:47

Copilot started reviewing on behalf of kamja0510 January 28, 2026 12:48 View session

Copilot AI reviewed Jan 28, 2026

View reviewed changes

kamja0510 merged commit 469eb01 into develop Jan 29, 2026
7 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

[Feat] 1차 QA 기능 추가#90

[Feat] 1차 QA 기능 추가#90
kamja0510 merged 31 commits intodevelopfrom
feat/#89-qa-1-new-feature

kamja0510 commented Jan 28, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Jan 28, 2026

Uh oh!

Copilot AI Jan 28, 2026

Uh oh!

Copilot AI Jan 28, 2026

Uh oh!

Copilot AI Jan 28, 2026

Uh oh!

Copilot AI Jan 28, 2026

Uh oh!

Copilot AI Jan 28, 2026

Uh oh!

Copilot AI Jan 28, 2026

Uh oh!

Copilot AI Jan 28, 2026

Uh oh!

Copilot AI Jan 28, 2026

Uh oh!

Copilot AI Jan 28, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Comments

Conversation

kamja0510 commented Jan 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Related issue 🛠

Work Description ✏️

Screenshot 📷

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 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

kamja0510 commented Jan 28, 2026 •

edited

Loading