fix(iOS): Allow the original delegate to handle non-Notifee notifications first

[gotcha84]

Issues

Fixes

• iOS: Own userNotificationCenter:didRecieveNotificationResponse:withCompletionHandler is overwritten #925
• iOS: Own userNotificationCenter:didRecieveNotificationResponse:withCompletionHandler is overwritten #984

Overview

For notifications received when the app is backgrounded or killed in iOS (through userNotificationCenter:didReceiveNotificationResponse:), the original delegate is not invoked for non-Notifee notifications. The existing code attempts to parse the unknown notification into a format Notifee understands before giving the original delegate a chance to handle it.

I've moved up original delegate so that it handles a non-Notifee notification first (if it exists). If there is no original delegate, then we fall back to parsing the unknown notification and allowing Notifee to handle it.

Testing

1. Jest tests all pass
2. iOS end-to-end tests all pass
3. I have an app that handles both Iterable and Notifee notifications.
   1. Before this change: The Iterable notification was being handled by Notifee, resulting in a default behavior of launching to the app home screen.
   2. After this change: Notifee falls back to the original delegate when encountering the Iterable notification, resulting in handling the notification correctly & launching the app to the correct screen.

[CLAassistant]

All committers have signed the CLA.

[BMR11]

@mikehardy, Just wondering if we can get this in as this may be more common case and the bug prevents one listener from other

[SMPinCode]

Tried this.
messaging().getInitialNotification() works now when iOS app is terminated.

[github-actions]

Hello 👋, this PR has been opened for more than 2 months with no activity on it.

If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing!

You have 15 days until this gets closed automatically

[fobos531]

not stale

@mikehardy Is there a possibility to get this reviewed and merged? It would really help for projects that use libraries that depend on Notifee, but for projects where advanced notifee features are not directly required by the consumer (the developer), instead react-native-firebase is enough. This is exactly the use case I have and would greatly appreciate this change being merged. (same as in android, but that's a story for another PR)

[carlbleick]

If I correctly understand these changes, wouldn't the original delegate always get called instead of the notifee handler?
E.g when I use react-native-firebase?
Should there be an option to toggle that behaviour?

[gezquinndesign]

Good work @gotcha84 ! Looking forward to seeing this merged.

[github-actions]

Hello 👋, this PR has been opened for more than 2 months with no activity on it.

If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing!

You have 15 days until this gets closed automatically

[jacobmllr95]

I can confirm that this solves #913 as well.

I would love to see this merged soon!

[github-actions]

Hello 👋, this PR has been opened for more than 2 months with no activity on it.

If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing!

You have 15 days until this gets closed automatically

[fobos531]

not stale