Skip to content

Comments

Feat/issue #31 - 알림 subText 고려#32

Merged
Richter3766 merged 9 commits intomainfrom
Feat/issue-#31
Mar 1, 2025
Merged

Feat/issue #31 - 알림 subText 고려#32
Richter3766 merged 9 commits intomainfrom
Feat/issue-#31

Conversation

@Richter3766
Copy link
Owner

@Richter3766 Richter3766 commented Mar 1, 2025

PR

작업 내용

  • 카톡 단톡방과 같이 subText가 있는 경우 해당 값으로 묶어 리스트로 보여줌.
  • 위 수정에 따라 라우팅 경로 수정
  • 버그 수정

참고 사항

✏ Git Close

#31

@Richter3766
feat: title에 subText 요소를 DAO에 적용
0c5d0da 
빠른 쿼리를 위한 인덱스 추가 및 DB 마이그레이션
@Richter3766
feat: 도메인 계층에 subText 변경 적용
43fdb84
@Richter3766
feat: title 리스트에서 title 인자 삭제
c11aab3 
특정 앱의 제목 리스트를 가져오는 데
특정 title이 담겨있으면 안됨.
없어야 title 별로 하나씩 가져올 수 있음.
@Richter3766
feat: UI, subText 존재 여부에 따라 스크린 적용
8c95ca4
@Richter3766
fix: title -> noti 라우팅 경로 수정
12f7709
@Richter3766
fix: 알림인데 subText가 나오던 현상 수정
05e3fcb 
알림의 제목은 title이 와야 맞음.
@Richter3766
refactor: dao 쿼리 개선
cd997c5 
우선순위와 비우선순위 리스트 쿼리의 경우
priority 말고 다른 점이 없으므로 하나의 함수로 만듦.
@Richter3766
refactor: 불필요 import 문 삭제
732f868
@Richter3766
Test: subText 컬럼 추가에 따른 테스트 수정
affceec
@Richter3766 Richter3766 added the enhancement New feature or request label Mar 1, 2025
@Richter3766 Richter3766 self-assigned this Mar 1, 2025
@Richter3766 Richter3766 linked an issue Mar 1, 2025 that may be closed by this pull request
feat: 알림 subText 고려 #31
Closed
2 tasks
@github-actions
Copy link

github-actions bot commented Mar 1, 2025

Code Coverage

File Coverage [74.80%] ✅
app/src/main/java/com/example/notimanager/data/dto/NotificationDto.kt 100.00% ✅
app/src/main/java/com/example/notimanager/data/dto/NotificationTitleDto.kt 100.00% ✅
app/src/main/java/com/example/notimanager/data/model/NotificationModel.kt 85.71% ✅
app/src/main/java/com/example/notimanager/data/repository/NotificationAppRepository.kt 100.00% ✅
app/src/main/java/com/example/notimanager/data/repository/NotificationDomainRepository.kt 60.00% ✅
app/src/main/java/com/example/notimanager/data/repository/NotificationTitleRepository.kt 100.00% ✅
app/src/main/java/com/example/notimanager/domain/model/Notification.kt 100.00% ✅
app/src/main/java/com/example/notimanager/domain/model/NotificationTitle.kt 100.00% ✅
app/src/main/java/com/example/notimanager/domain/usecase/NotificationTitleUseCase.kt 50.00% ✅
app/src/main/java/com/example/notimanager/domain/usecase/NotificationUseCase.kt 75.00% ✅
app/src/main/java/com/example/notimanager/domain/utils/AppIconGetter.kt 0.00% ❌
Total Project Coverage 53.02% ✅

@Richter3766 Richter3766 merged commit 417c219 into main Mar 1, 2025
1 check passed
@Richter3766 Richter3766 deleted the Feat/issue-#31 branch March 1, 2025 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@Richter3766 Richter3766

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat: 알림 subText 고려

1 participant

@Richter3766