-
Notifications
You must be signed in to change notification settings - Fork 9
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
[feature/#946] WebView에 사진 다운로드 기능 추가 #947
base: develop
Are you sure you want to change the base?
Conversation
|
12e16a4
to
4e66166
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
웹뷰 처리 재밌네요~
조만간 정독하겠슴다
그리고.. 저 다이얼로그는 제가 해볼게요 ㅜ.ㅜ
downloadManager.enqueue(request) | ||
} else { | ||
permissionRequestLauncher.launch(android.Manifest.permission.WRITE_EXTERNAL_STORAGE) | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요거 특정 횟수(아마 3회) 이상 요청하면 더이상 팝업 안뜨는걸로 아는데 그거 관련 로직 추가하는건 어떨까요??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 제 기기에서 테스트가 안되는데 ㅋㅋㅋ 어카죠..
4291d92
to
88063c5
Compare
private val permissionRequestLauncher = registerForActivityResult( | ||
ActivityResultContracts.RequestPermission() | ||
) { isGranted -> | ||
if (!isGranted) { | ||
toast("권한을 받아오지 못했습니다.") | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 부분 홈에서 알림 권한 얻어올 때도 쓰던데 따로 분리할 수 있을까요?
binding.webView.setDownloadListener { url, userAgent, contentDisposition, mimetype, _ -> | ||
if (ContextCompat.checkSelfPermission( | ||
this, | ||
android.Manifest.permission.WRITE_EXTERNAL_STORAGE | ||
) == android.content.pm.PackageManager.PERMISSION_GRANTED |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요 부분 아래 함수들(~handle)과 달리 따로 함수화하지 않은 이유가 있으실까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다~!
혹시 시연 영상 있으면 PR에 첨부 가능할까요 !?
e76e2a3
to
be3ca64
Compare
806a6d9
to
403552f
Compare
403552f
to
f892428
Compare
f892428
to
848ff2c
Compare
What is this issue?