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

Feat : 알림 권한 설정 다이어로그 #226

Merged
merged 7 commits into from
Apr 2, 2024

Conversation

huiwoo-jo
Copy link
Contributor

📮 관련 이슈

✍️ 구현 내용

  • 알림 권한이 부여되지 않을 경우 알림 권한 설정에 대한 다이어로그를 띄웁니다.
    • 온보딩 3단계의 알림 설정
    • 설정 화면의 알림 스위치 : 학과 / 키워드
  • 커스텀 다이어로그를 제작하였습니다.
    • 취소 : 다이어로그 창이 종료됩니다.
    • 설정 화면 이동 : DMforU의 어플 권한 설정 창으로 이동합니다.
  • ⭐ 설정 화면의 학과 / 키워드의 알림 설정이 켜진 상태로 어플의 알림 권한을 취소할 경우 스위치는 종료되나 알림 권한 설정이 여러차례 뜨는 것을 수정할 필요가 있습니다.

📷 구현 영상

사진
image

영상

온보딩 3단계 설정 화면(권한 off -> 설정 on) 설정 화면 (설정 on -> 권한 off)
https://github.com/TeamDMU/DMU-Android/assets/84004687/c5e74cf4-990c-4943-ad9a-c129691d2325 https://github.com/TeamDMU/DMU-Android/assets/84004687/2dec8a64-efae-48fe-acb2-1194727a2bb8 https://github.com/TeamDMU/DMU-Android/assets/84004687/2a4d5b36-fa6a-43cf-a95b-0adb6b99a5ee

✔️ 확인 사항

  • 컨벤션에 맞는 PR 타이틀
  • 관련 이슈 연결
  • PR 관련 정보 연결 (작업자, 라벨, 마일스톤 등)
  • Github Action 통과

@huiwoo-jo huiwoo-jo added 🎨 DESIGN User interface design 🤗 FEATURE Develop this project 🩵희우 labels Apr 2, 2024
@huiwoo-jo huiwoo-jo requested review from hoyahozz and m6z1 April 2, 2024 12:55
@huiwoo-jo huiwoo-jo self-assigned this Apr 2, 2024
Copy link
Collaborator

@m6z1 m6z1 left a comment

Choose a reason for hiding this comment

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

바로 머지 고고

@huiwoo-jo huiwoo-jo merged commit e069868 into develop Apr 2, 2024
1 check passed
@huiwoo-jo huiwoo-jo deleted the feature/onboarding-dialog branch April 2, 2024 15:06
Copy link
Collaborator

@m6z1 m6z1 left a comment

Choose a reason for hiding this comment

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

굿굿

Comment on lines 12 to +13


Copy link
Collaborator

Choose a reason for hiding this comment

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

두 줄 공백은 나중에 코드 리팩할 때 하나 씩 확인하면서 지워가면 좋을 것 같네용

@@ -111,6 +117,24 @@ class SettingFragment : BaseFragment<FragmentSettingBinding, SettingViewModel>()
}
}

fun checkPermission(){
Copy link
Collaborator

Choose a reason for hiding this comment

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

외부에서도 사용이 되는 메서드인가용 ?

Comment on lines +26 to +36
// 제목, 내용 설정
binding.customTvTitle.text = title
binding.customTvContent.text = content

// 취소 버튼
binding.customTvBtn1.setOnClickListener {
dismiss()
}
// 확인 버튼
binding.customTvBtn2.setOnClickListener {
val intent =
Copy link
Collaborator

Choose a reason for hiding this comment

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

나중에 로직 분리하면 좋을 것 같아요!
그리고 onViewCreated 와 onCreatedView에서도 알아보심 좋을 듯 합니다 << 프레그먼트 생명주기

android:background="@drawable/bg_stroke_gray300_radius_2dp">

<TextView
android:id="@+id/customTvTitle"
Copy link
Collaborator

Choose a reason for hiding this comment

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

id 컨벤션이 다른 것 같아요

android:layout_width="wrap_content"
android:layout_height="0dp"
android:layout_marginTop="24dp"
android:text="제목"
Copy link
Collaborator

Choose a reason for hiding this comment

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

xml에서 위치를 확인하고 싶은 것이라면
hint:text="제목"
을 사용하시면 되구 사용하지 않는 text값은 지워줍시다용

android:layout_marginTop="20dp"
android:layout_marginEnd="28dp"
android:layout_marginBottom="20dp"
android:text="취소"
Copy link
Collaborator

Choose a reason for hiding this comment

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

그리고 string 추출도 !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 DESIGN User interface design 🤗 FEATURE Develop this project 🩵희우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

알림 설정 다이어로그
2 participants