braze-inc / braze-swift-sdk

Braze SDK for the Apple ecosystem, including: iOS, macOS, iPadOS, visionOS, tvOS
https://www.braze.com
Other
52 stars 19 forks source link

[Feature]: Enable filtering push notifications #94

Closed PlanesWalkerr closed 4 months ago

PlanesWalkerr commented 12 months ago

What problem are you facing?

I want to filter some push notifications according to Apple documentation. Subclassing from BrazeNotificationService.NotificationService in NotificationServiceExtension doesn't allow me to override UNNotificationServiceExtension.didReceive(_:withContentHandler:) method. Is there any way to filter push notifications?

Workarounds

Seems like it's not possible in current version.

Ideal Solution

Maybe add optional handler where an app can decide if push notification should be shown

Other Information

No response

jerielng commented 10 months ago

Hey @PlanesWalkerr, it sounds like you're trying to add your own custom logic in UNNotificationServiceExtension.didReceive(_:withContentHandler:), is that correct?

The Braze implementation of that method calls the brazeHandle method, which is publicly available. What you could do instead of directly inheriting from BrazeNotificationService.NotificationService is to simply implement UNNotificationServiceExtension yourself, and when you override didReceive(_:withContentHandler:), you can call the brazeHandle method, along with your own custom logic.

The minimal implementation on that page should describe how the method would look like, but in summary, it would be something like this:

class YourNotificationService: UNNotificationServiceExtension {
  override func didReceive(
  _ request: UNNotificationRequest,
  withContentHandler contentHandler: @escaping (UNNotificationContent) -> Void
) {
  if brazeHandle(request: request, contentHandler: contentHandler) {
    return
  }

  // Your custom logic

  contentHandler(request.content)
 }
}

Let us know if that implementation works for your use case, and if so, feel free to close this issue! Thank you!

lowip commented 4 months ago

Hi @PlanesWalkerr,

We're closing this ticket due to inactivity, feel free to re-open it if you're still facing this issue.

Best,