-
Notifications
You must be signed in to change notification settings - Fork 74
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
[SDK-1970] - Push primer for Android 13 #351
Conversation
…ST_NOTIFICATION permission, update targetSDK/compileSDK version for sample app
…app(half-interstitial, alert type)
…itial & alter type)
…va to directly call hard permission request dialog
…permission flow from HTML fragment
…Primer # Conflicts: # gradle/wrapper/gradle-wrapper.properties # versions.properties
…ask/SDK-1769/PushPrimer
…age support in local IAM
…AppController to accept only jsonObj for creating local IAM
…ppBuilder`,navigation to settings dialog flow, Remove unused code in `CTInAppNotificationButton` and `CTInAppNotificationMedia`
…, refactor CleverTapAPI.java, AlertDialogPromptForSettings.kt and InAppController.java, Update sample,
….kt, refactor hard permission flow if permission is denied
…fix string resource keys with ct
…ask/SDK-1769/PushPrimer # Conflicts: # clevertap-core/src/main/java/com/clevertap/android/sdk/InAppNotificationActivity.java
…Primer # Conflicts: # clevertap-core/src/main/java/com/clevertap/android/sdk/CleverTapAPI.java
…ask/SDK-1769/PushPrimer
…ppress lint for Android 13
…ppBaseFragment.java and CTInboxListViewFragment.java, Refactor push permission response listener
…ap/clevertap-android-sdk into task/SDK-1769/PushPrimer
…cation permission.
…ask/SDK-1769/PushPrimer # Conflicts: # clevertap-core/src/main/java/com/clevertap/android/sdk/InAppNotificationActivity.java # clevertap-core/src/main/java/com/clevertap/android/sdk/inbox/CTInboxActivity.java
# Conflicts: # README.md # docs/CTGEOFENCE.md # docs/CTPUSHTEMPLATES.md # versions.properties
clevertap-core/src/main/java/com/clevertap/android/sdk/BaseCallbackManager.java
Outdated
Show resolved
Hide resolved
clevertap-core/src/main/java/com/clevertap/android/sdk/InAppNotificationListener.java
Show resolved
Hide resolved
clevertap-core/src/main/java/com/clevertap/android/sdk/PushPermissionManager.java
Show resolved
Hide resolved
clevertap-core/src/main/java/com/clevertap/android/sdk/inapp/AlertDialogPromptForSettings.kt
Show resolved
Hide resolved
clevertap-core/src/main/java/com/clevertap/android/sdk/inapp/CTLocalInApp.kt
Show resolved
Hide resolved
* `promptForPushPermission(boolean showFallbackSettings)` | ||
*New callback `PushPermissionResponseListener` available which returns after user Allows/Denies notification permission. | ||
* From 4.7.0+ existing callback `InAppNotificationListener` will now have `onShow(CTInAppNotification)` method which needs to implemented. | ||
|
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.
@william-ct Give a link to Usage.md in this changelog
…n from app's notification settings
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.
LGTM
…javadoc for new classes
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.
LGTM
# Conflicts: # README.md # clevertap-core/src/main/java/com/clevertap/android/sdk/inbox/CTInboxActivity.java # docs/CTCORECHANGELOG.md # docs/CTGEOFENCE.md # docs/CTPUSHTEMPLATES.md # templates/CTCORECHANGELOG.md # versions.properties
isPushPermissionGranted()
promptPushPrimer(JSONObject)
promptForPushPermission(boolean showFallbackSettings)
PushPermissionResponseListener
available which returns after the user Allows/Denies notification permission.InAppNotificationListener
will now haveonShow(CTInAppNotification)
method which needs to implemented.