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 push notification methods & update delegate re foreground notification (iOS only) #71

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rvencio880
Copy link

This will implement the following changes:

  1. Add an iOS only method -- hasUserAnsweredNotificationPrompt -- that gets a boolean indicating if the user received and answered (allowing or not) the prompt to allow notifications
  2. Add an iOS only method -- registerPushNotification -- for prompting for permission to send push notification if the user was never prompted with it
  3. Update delegate method -- userNotificationCenter:didReceiveNotificationResponse:withCompletionHandler -- to clear the 'applicationIconBadgeNumber' once user notification was sent from Braze. This ensures that foreground notifications are being delivered

update delegate to ensure foreground notifications are being delivered
@santiq
Copy link

santiq commented Feb 16, 2022

@Bucimis
Copy link
Collaborator

Bucimis commented Feb 16, 2022

Hi @rvencio880, thanks for this contribution. Updating our iOS + Cordova push integration for the future is part of our 2022 roadmap and will be built on top of our upcoming Swift SDK. We plan to significantly upgrade automation, customization, and flexibility in the new world.

Let us know if you'll be able to use this fork for your needs until then.

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.

3 participants