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

fix : 잘못된 권한 대응 수정 #70

Merged
merged 1 commit into from
Jul 23, 2024
Merged

fix : 잘못된 권한 대응 수정 #70

merged 1 commit into from
Jul 23, 2024

Conversation

jowunnal
Copy link
Contributor

@jowunnal jowunnal commented Jul 23, 2024

📌 Issue Number

📘 작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리팩토링
  • 디자인 수정

📙 작업 내역

구현 내용 및 작업 내역을 기재합니다.

  • api 30 이하에서 Activity. onRequestPermissionsResult 를 오버라이딩 하여 권한 요청의 결과를 처리하는 방법이 deprecation 되었기 때문에 31 부터는 ActivityResult*** 를 이용하여 권한 또는 화면을 요청을 하게 되는데, 해당 부분을 잘못 이해하여 하위버전을 위해서 하위버전인 경우 onRequestPermissionsResult 를 오버라이딩 하여 처리해야 한다고 생각하고 작업했던 부분을 수정
  • 내부적으로 권한 또는 화면 요청의 로직이 ActivityCompat 의 함수로 처리되기 때문에 개발자가 신경써야 할 부분이 없음
  • 따라서, ActivityResultLauncher, ActivityResultContract 를 이용하여 권한을 처리하고 compose 용도로 지원하는 androidx.activity.compose.ActivityResultRegistry.kt 의 함수들을 이용하면 됨

📋 체크리스트

  • PR 제목에 Prefix(Feat, Refactor, Fix...etc) 와 작업 번호 및 내용을 요약 기재
  • 사용되지 않은 import 문 제거 & code reformatting 수행
  • PR과 관련있는 코드만 추가
  • 코드 개선 및 검토 완료



- api 30 이하에서 Activity. onRequestPermissionsResult 를 오버라이딩 하여 권한 요청의 결과를 처리하는 방법이 deprecation 되었기 때문에 31 부터는 ActivityResult*** 를 이용하여 권한 또는 화면을 요청을 하게 되는데, 해당 부분을 잘못 이해하여 하위버전에서 onRequestPermissionsResult 를 오버라이딩 하여 처리해야 한다고 생각하고 작업했던 부분을 수정
- 내부적으로 권한 또는 화면 요청의 로직이 ActivityCompat 의 함수로 처리되기 때문에 개발자가 신경써야 할 부분이 없음
- 따라서, ActivityResultLauncher, ActivityResultContract 를 이용하여 권한을 처리하고 compose 용도로 지원하는 androidx.activity.compose.ActivityResultRegistry.kt 의 함수들을 이용하면 됨
@jowunnal jowunnal self-assigned this Jul 23, 2024
@jowunnal jowunnal requested a review from kyungil9 July 23, 2024 09:29
@jowunnal jowunnal added the hot fix 버그 발생시 핫 픽스 관련 작업 label Jul 23, 2024
@jowunnal jowunnal merged commit 6c5fa3a into develop Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hot fix 버그 발생시 핫 픽스 관련 작업
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants