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 requestPermissions did not exist pre M #89

Merged
merged 2 commits into from
May 12, 2016
Merged

Conversation

supermao
Copy link

No description provided.

distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-2.5-all.zip
distributionUrl=https\://services.gradle.org/distributions/gradle-2.10-all.zip
Copy link
Contributor

Choose a reason for hiding this comment

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

make it 2.13

@@ -69,12 +68,15 @@ static boolean shouldShowRequestPermissionRationale(Activity activity, String...
*
* @param airMapInterface the callback interface if permission is granted.
*/
@TargetApi(Build.VERSION_CODES.M)
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you still need this

Copy link
Author

Choose a reason for hiding this comment

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

Nope, don't need this anymore, plus this was misleading as well..

@felipecsl
Copy link
Contributor

👍

@felipecsl felipecsl merged commit fb194a0 into airbnb:master May 12, 2016
@sreejithraman
Copy link

seeing as the library wasn't updated in a while, can we push an update with this? It is currently affecting my app and would be nice to update to the new version.

@felipecsl
Copy link
Contributor

@sreejithraman for sure. I'm cutting 1.3.5 now...

@felipecsl
Copy link
Contributor

@sreejithraman this has been released as version 1.4.0

@sreejithraman
Copy link

@felipecsl perfect! thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants