Handle UNUserNotificationCenter completionHandlers

[blankg]

Summary

Currently only didReceiveRemoteNotification: fetchCompletionHandler: are handle async by the JS complete method. This allows some time to perform tasks in background however the userNotificationCenter: didReceiveNotificationResponse: withCompletionHandler completion handler is called immediately which means not much time to perform tasks in background (about 1 sec from my experience) and calling complete from JS is meaningless.

This enhancement handles the NSUserNotificationCenter completionHandler in a similar way the fetchCompletionHandler is handled.

Checklist

• Supports Android
• [x ] Supports iOS
• e2e tests added or updated in /tests/e2e/*
• Updated the documentation in the docs repo
  • LINK TO DOCS PR HERE
• Flow types updated
• Typescript types updated

Test Plan

Release Plan

[IOS][ENHANCEMENT] [NOTIFICATIONS] - handle NSUserNotificationCenter completion handlers

---

Think react-native-firebase is great? Please consider supporting the project with any of the below:

• 👉 Donate via Open Collective
• 👉 Follow React Native Firebase and Invertase on Twitter
• 👉 Star this repo on GitHub ⭐️
• 👉 Contribute; see our contributing guide

[codecov]

Codecov Report

Merging #1803 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1803   +/-   ##
=======================================
  Coverage   76.11%   76.11%           
=======================================
  Files          72       72           
  Lines        1838     1838           
=======================================
  Hits         1399     1399           
  Misses        439      439

[codecov]

Codecov Report

Merging #1803 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1803   +/-   ##
=======================================
  Coverage   76.11%   76.11%           
=======================================
  Files          72       72           
  Lines        1838     1838           
=======================================
  Hits         1399     1399           
  Misses        439      439

[codecov]

Codecov Report

Merging #1803 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1803   +/-   ##
=======================================
  Coverage   76.11%   76.11%           
=======================================
  Files          72       72           
  Lines        1838     1838           
=======================================
  Hits         1399     1399           
  Misses        439      439

[codecov]

Codecov Report

Merging #1803 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1803      +/-   ##
==========================================
+ Coverage   76.11%   76.16%   +0.05%     
==========================================
  Files          72       72              
  Lines        1838     1838              
==========================================
+ Hits         1399     1400       +1     
+ Misses        439      438       -1

[Salakar]

@blankg thanks for making those changes and replying to my points. It all makes sense now, sorry about the confusion 🙈

I'll go ahead and merge this now for the v5.2.0 release.

---

Loving react-native-firebase and the support we provide? Please consider supporting us with any of the below:

• 👉 Back financially via Open Collective
• 👉 Follow React Native Firebase and Invertase on Twitter
• 👉 Star this repo on GitHub ⭐️