Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notification 기반 잡기 #20

Merged
merged 17 commits into from
Jan 26, 2025
Merged

Notification 기반 잡기 #20

merged 17 commits into from
Jan 26, 2025

Conversation

nohjunh
Copy link
Member

@nohjunh nohjunh commented Jan 26, 2025

Notification 기능을 위한 기반 작업입니다.

  • 퍼미션 허용받기와는 별개로 Notification 생성 기능을 작업하기 위한 core:notifications 모듈 생성
    • 퍼미션 허용여부와 목표알림 on/off 여부는 유저 설정과 연관이 있다고 생각해 생성 기능과는 분리
  • ios와의 동작을 동일하게 가져가기 위한 정책에 따른 작업
    • notificationsPermissionState의 shouldShowRationale의 여부를 따져서 나중에 새로운 시스템 팝업을 띄우지 않도록 정책이 정해졌기에, 최초 대시보드 진입 시에만 시스템 팝업을 통한 Goal Notification Checked 상태 갱신
    • 추후에 다시 Notification에 대한 시스템 팝업을 띄우지 않도록 isSystemNotificationDialogDisabled 값을 User proto datastore에서 관리
    • DobeDobe 커스텀 다이얼로그 -> 확인 클릭 -> 시스템 Notification 퍼미션 허용/거절 -> 거절 시 이후에는 셋팅PR에서 앱 설정으로 곧바로 랜딩하도록 처리 예정
  • DobeDobe 다이얼로그 기반 잡기

Copy link
Member

@murjune murjune left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

작업타임에 말했던 의견 때문에, 작업하느라 고민이 더 많이 되셨을 것 같네요 😵‍💫
고생 많으셨습니다!

Copy link
Member

@murjune murjune left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

주말까지 고생 많으셨습니다~!!

* feat : Implement base for feature:setting

* feat : Connect dashboard and settings navigation

* feat : add temporary wrapping structure for Switch

* feat : add openPlayStore Util

* feat : add SettingTopAppBar component

* feat : add SettingRow component

* feat : Implement settings feature requirements

* refactor : lint

* refactor : Rename onSettingClick to navigateToSetting

* refactor : Encapsulate settings UI elements within SettingBody

* refactor : Rename isSystemNotificationEnabled to updatedSystemNotificationEnabled

* refactor : Remove Box and use modifiers for layout
@nohjunh nohjunh merged commit 539ec97 into develop Jan 26, 2025
1 check passed
@nohjunh nohjunh self-assigned this Jan 26, 2025
@nohjunh nohjunh deleted the feat/notification-base branch January 28, 2025 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants