-
Notifications
You must be signed in to change notification settings - Fork 560
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
Android API level 22 crashes on getAll()
#331
Comments
+1 |
yes if you can submit a PR that supports older versions of android and maintains support for new versions of android as well that would be appreciated @platonish FWIW supporting older phone os versions is not a priority for this project. |
Hi @morenoh149, sorry for the late response. Actually on second thought, I think it would be better if we simply added a note in the readme regarding this, so that those who don't require it aren't requesting unnecessary permissions. LMK what you think. |
@platonish yes absolutely. If you open PR for the readme I'll merge. Basically, you're suggesting we also list |
@morenoh149 yup pretty much! Take a look 🙂 |
Hi @morenoh149, i have same problem :/ and added the permissions but this not working :/. Do i have to do something else? |
@ssancheza227 are you having issues on both platforms? usually the issue is not following the instructions correctly, read the whole readme first then try to install it. |
@morenoh14 I have the problem only in android but only in versions custom for example v5.1.1 in this version the app crash but in v5.0 the app work, maybe i have the force the version in build.gradle :/. |
@ssancheza227 I've had the same problem before |
@jamalhassouni Great! you would shared your solution in code? please :) |
Whenever I tried to access contacts through the
getAll()
method on a Nexus 4 running Android 5.1.1 Lollipop (API level 22), the app would immediately crash without warning. I checked logcat, and found out that a permission forREAD_PROFILE
was also required. I added that to AndroidManifest.xml and everything worked fine.Also, the permission granted status is represented as a boolean
true
value rather than the string'granted'
. This is also a condition that needs to be handled separately for older versions of Android (<= API level 22).I can create a quick PR for handling this case.
The text was updated successfully, but these errors were encountered: