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

Add requiresUserPrivacyConsent() getter #598

Merged
merged 2 commits into from
Jul 25, 2018
Merged

Conversation

Nightsd01
Copy link
Contributor

@Nightsd01 Nightsd01 commented Jul 23, 2018

• Since developers can manually require consent by editing the manifest, the wrapper SDK's need a way of knowing if the Android SDK is waiting on the user's privacy consent
• Exposes a public getter that returns TRUE if consent is required but hasn't been provided yet.


This change is Reviewable

• Since developers can manually require consent by editing the manifest, the wrapper SDK's need a way of knowing if the Android SDK is waiting on the user's privacy consent
• Exposes a public getter that returns TRUE if consent is required but hasn't been provided yet.
@Nightsd01 Nightsd01 requested a review from jkasten2 July 23, 2018 21:40
• Adds a testcase to make sure that the new requiresUserPrivacyConsent() method returns correct results
Nightsd01 added a commit to OneSignal/OneSignal-Flutter-SDK that referenced this pull request Jul 23, 2018
• Added a new getter method in the android native SDK (pull request = OneSignal/OneSignal-Android-SDK#598)
• This new method returns a boolean indicating if the SDK is waiting for user privacy consent before it initializes
• This new getter method makes the behavior for `requiresUserPrivacyConsent()` the same as it is implemented in iOS
Copy link
Member

@jkasten2 jkasten2 left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 2 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@Nightsd01 Nightsd01 merged commit af55a25 into master Jul 25, 2018
@Nightsd01 Nightsd01 deleted the public_requires_consent branch July 25, 2018 01:23
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.

2 participants