-
Notifications
You must be signed in to change notification settings - Fork 4
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
Request permission for getting GPSI (Consumption Reporting - Generating the reportingClientId #57) #33
Conversation
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.
@shilinding Please check my comments and adjust the implementation accordingly
..._5GMSdAwareApplication/app/src/main/java/com/fivegmag/a5gmsdawareapplication/MainActivity.kt
Outdated
Show resolved
Hide resolved
..._5GMSdAwareApplication/app/src/main/java/com/fivegmag/a5gmsdawareapplication/MainActivity.kt
Outdated
Show resolved
Hide resolved
var needReqPermission: Boolean = false | ||
var idx: Int = 0 | ||
val permissions = arrayOf( | ||
Manifest.permission.ACCESS_FINE_LOCATION, |
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.
Initialize an empty array
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.
I also intend to implement so before, but I don't know Android support empty array .
Done
..._5GMSdAwareApplication/app/src/main/java/com/fivegmag/a5gmsdawareapplication/MainActivity.kt
Outdated
Show resolved
Hide resolved
..._5GMSdAwareApplication/app/src/main/java/com/fivegmag/a5gmsdawareapplication/MainActivity.kt
Outdated
Show resolved
Hide resolved
..._5GMSdAwareApplication/app/src/main/java/com/fivegmag/a5gmsdawareapplication/MainActivity.kt
Outdated
Show resolved
Hide resolved
…use dynamic array)
…aviod 3 warnings)
…der #51 (#37) * test first commit * * Consumption Reporting - Generating the reportingClientId-issue #57: GPSI * Consumption Reporting - Request permission for getting GPSI #33 * Consumption Reporting - Request permission for getting GPSI #33 * Consumption Reporting - Request permission for getting GPSI #33(use dynamic array) * Consumption Reporting - Request permission for getting GPSI #33(aviod 3 warnings) * Support for the expires header #51: support default timer period value be configured in configuration file; refactor code * Support for the expires header #51: move default timer configuration file to Media Session Handler
#35