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 [#70] 권한 허용 #79

Merged
merged 11 commits into from
Jan 14, 2024
Merged

Feat [#70] 권한 허용 #79

merged 11 commits into from
Jan 14, 2024

Conversation

Zoe0929
Copy link
Member

@Zoe0929 Zoe0929 commented Jan 13, 2024

👾 작업 내용

스크린타임과 푸시 알림 권한 설정을 하였습니다.

🚀 PR Point

  • 스크린타임 설정 후 알림 설정을 해야하는데, 스크린타임은 비허용이어도 다시 요청이 가능한 반면, 푸시는 다시 요청이 불가했습니다.
    그래서 푸시 알림 설정 요청 -> 스크린 타임 설정 요청 을 했으나 중간에 겹치는 과정이 있습니다. 해결 방법 더 찾아볼 예정입니다!
  • 푸시 알림 요청이 다시 불가하므로, 버튼을 disable할 시 사용자는 온보딩 프로세스를 더 이상 진행 불가합니다. 따라서, 버튼을 disable하는 것은 스크린타임 권한 요청이 없을 때로 한정합니다
  • 권한 허용 함수를 Global로 만들어, TabBar에 속한 ViewController에 접근할 때마다 권한을 확인합니다.
    func requestPermision() {
    requestAuthNoti()
    requestAuthorization()
    }

📸 스크린샷

구현 내용 스크린샷
스크린타임만 허용 Simulator Screen Recording - iPhone 13 mini - 2024-01-14 at 08 49 30
푸시알림만 허용 Simulator Screen Recording - iPhone 13 mini - 2024-01-14 at 08 55 14
모두 허용 Simulator Screen Recording - iPhone 13 mini - 2024-01-14 at 08 37 43

✅ Issue

Resolved #70

@Zoe0929 Zoe0929 added 🐰지희 지희의 issue 🌈 feat 기능 구현 labels Jan 13, 2024
@Zoe0929 Zoe0929 added this to the 🚀1차 스프린트🚀 milestone Jan 13, 2024
@Zoe0929 Zoe0929 self-assigned this Jan 13, 2024
@Zoe0929 Zoe0929 linked an issue Jan 13, 2024 that may be closed by this pull request
Copy link
Member

@kim-seonwoo kim-seonwoo left a comment

Choose a reason for hiding this comment

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

Zoe님 수고하셨습니다!!
LGTM👀

Comment on lines +21 to +23
override func viewWillAppear(_ animated: Bool) {
super.viewWillAppear(animated)
}
Copy link
Member

Choose a reason for hiding this comment

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

p5. 혹시 이거 생성하신 이유를 알 수 있을까요??

Copy link
Contributor

Choose a reason for hiding this comment

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

네이스~

Copy link
Member Author

Choose a reason for hiding this comment

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

이건 지우도록 하겠습니다!

Copy link
Contributor

@boyeon0119 boyeon0119 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 16 to 18
if authorizationCenter.authorizationStatus == .approved {

} else {
Copy link
Contributor

Choose a reason for hiding this comment

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

p5. 여기는 왜 비어잇는 걸까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

false로 분기처리 바꿨습니다!

Comment on lines +21 to +23
override func viewWillAppear(_ animated: Bool) {
super.viewWillAppear(animated)
}
Copy link
Contributor

Choose a reason for hiding this comment

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

네이스~

@Zoe0929 Zoe0929 merged commit b16eb9a into develop Jan 14, 2024
@Zoe0929 Zoe0929 deleted the feat/#70-PermissionGranted branch January 15, 2024 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌈 feat 기능 구현 🐰지희 지희의 issue
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Feat] 스크린타임 / 푸시 알림 권한 설정
3 participants