Skip to content

Conversation

@DongJun-H
Copy link
Member

작업 사항

  • 기존 XML로 되어있던 알림 설정 화면에 대해 Compose로 마이그레이션
  • 기존 기기 내부에서 앱 알림 설정에 대해 SharedPrference 값으로 관리하던 부분에 대해 제거

참고

- Define types for each notification setting
- Implement notification settings by syncing local and server data
- Add and integrate an app notification settings dialog
- Refactor ViewModel to use internal variables instead of directly using preferences
- Maintain LiveData usage with `asLiveData` after renaming relevant variables
@DongJun-H DongJun-H requested a review from yuni-ju March 17, 2025 10:25
@DongJun-H DongJun-H self-assigned this Mar 17, 2025
@yuni-ju
Copy link
Member

yuni-ju commented Mar 26, 2025

기기의 알림 설정여부로 앱 알림 스위치 설정한 것 좋습니다!

이때 전체 알림 설정은 기기 설정과 앱 내부 설정여부가 잘 맞는데,
반응 알림은 맞지 않는 것 같습니다.

기기 설정 앱 설정
image image

이 기능은 의도하신 동작일까요?

@DongJun-H
Copy link
Member Author

DongJun-H commented Apr 2, 2025

해당 내용에 대해 다음과 같이 수정하였습니다.

  1. 알림 채널이 없는 경우 (notification channel 조회시 return null)
    • 현재 알림을 최초로 받은 이후에 알림 채널이 생성되도록 되어있습니다. 알림 채널을 최초로 생성하는 경우, 항상 알림을 킨 상태로 알림을 받도록 설정해 생성하기 때문에, 이 경우 서버의 설정값만 판단하면 됩니다.
  2. 알림 채널이 있고 켜져있는 경우
    • 알림 채널이 존재하는 경우, 알림 채널과 서버 설정값 모두 True 인 경우에만 true로 표시하도록 하였으며, 알림 채널은 켜져있는데, 서버 설정값은 false인 경우 true로 변경할때 서버 설정값만 조작하도록 하였습니다.
  3. 알림 채널이 있고 꺼져있는 경우
    • 알림 채널이 꺼져있는 경우, 기기 앱 설정 - 알림 설정 내부의 알림 카테고리로 바로 이동해, 설정하는 곳으로 이동하도록 하였습니다. 이때 서버 설정값보다 알림 카테고리 이동이 우선이기 때문에, 알림 채널이 true여도 돌아왔을때, 서버 설정값이 false이면 한번더 switch를 조작해 서버 설정값을 변경하도록 하였습니다.

반응 알림에 대한 조작은 모두 서버 설정값만 조작하도록 되어 있으며, 알림 카테고리에 대한 설정은 조회 및 이동만 하도록 처리하였습니다.

@yuni-ju
Copy link
Member

yuni-ju commented Apr 9, 2025

상태를 확인했는데 가끔씩 앱의 반응 설정은 꺼져있는데, 기기의 알림은 켜져있는 경우가 있습니다. 확인 부탁드립니닷

Apr-09-2025.21-58-22.mp4

===========>

앱 내 알림 설정이 off이고, 기기 설정상 앱 알림 권한이 on인 경우

👉 서버에서는 알림 요청을 보내지 않도록 처리하며,
기기의 알림 권한 여부와는 무관하게 앱 내 설정을 우선으로 함으로 확인했습니다!

@github-project-automation github-project-automation bot moved this from Todo to In Progress in DAYO 2.0 Apr 9, 2025
@DongJun-H DongJun-H merged commit 12c07ce into develop Apr 16, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in DAYO 2.0 Apr 16, 2025
@DongJun-H DongJun-H deleted the layout/issue-627 branch April 16, 2025 13:29
DongJun-H added a commit that referenced this pull request Feb 3, 2026
Migrate Setting Notification View from XML to Compose
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants