Just to add a note that I didn't update the example app use the flutter_local_notifications plugin like @bparrishMines did in https://github.com/FirebaseExtended/flutterfire/pull/1785/files. I imagine that was only done for testing purposes.

Any ETA when this one will be reviewed and merged?

If it's urgent, you could try referencing my fork in your app. See https://dart.dev/tools/pub/dependencies#git-packages in case you haven't referenced packages via git before. Would help knowing if it helps resolve issues for others as well beyond the testing I've done

@MaikuB thanks!

But I still not able to receive message and show local push notification when ios app is off, even with changes from your repo.

I update my pubspec with repo ref:

 firebase_messaging: 
    git:
      url: [email protected]:MaikuB/flutterfire.git
      path: packages/firebase_messaging/

also I add line to infoPlist:

	<key>FirebaseAppDelegateProxyEnabled</key>
	<false/>

To send msg I am using next cmd:

curl --location --request POST 'https://fcm.googleapis.com/fcm/send' \
--header 'Authorization: key=myApiKey' \
--header 'Content-Type: application/json' \
--data-raw '{
  "to": "/topics/myTopic",
  "data": {
  	"id":63
  }
}
'

So,

• android works as expected 👌
• when ios app is running - I am able to receive msg and show local notification.
• when ios app is off - nothing happen. But as soon as app started - it behaves like app just received a push notification.

@pro100svitlo can you post issues on my fork so we can make the PR thread for others easier to follow?

@pro100svitlo can you post issues on my fork so we can make the PR thread for others easier to follow?

done 👌

Thank you for the fix.
I got it to work.

On both iOS and Android it works for me now. I had to make a nother change though.
When receiving Notification while App is in foreground and sliding down the Notification Center the Notification in the Center gets removed.
Appearently this is intentional behaviour by firebase plugin. In the applicationDidBecomeActive the icon badges are set to 0 which triggers the removal.

- (void)applicationDidBecomeActive:(UIApplication *)application {
  _resumingFromBackground = NO;
...
//  application.applicationIconBadgeNumber = 1;
//  application.applicationIconBadgeNumber = 0;
}

I have tested the various app states on both iOS and Android and together with your fork the change doesn't seem to cause any issue.
So it solved it for me.

Maybe you can include it into this PR.

I thought Androids were supposed to dismiss notifications of apps on-screen. As logn as the callback is run, the user shouldn't have to have another notification in their tray.

@m-oehme there's an issue raised on this already #114. See thread for more info but not likely to be in this PR unless there's a response from the firebase team since they consider it to be working as intended for push notifications.

@Levi-Lesches not sure about Android. This was about iOS and there you normally have to remove them manually as a developer.

@MaikuB thank you, didn't see that one.

I'm not a top developer and surely not part of a big team such as Flutter... but is there a reason why @collinjackson or @kroikie haven't looked at this PR yet? it's been 27 days now and there's many of us waiting to include this plugin in our projects. imo it doesn't take more than a couple hours max to just take a look at the PR.

Not looking to sound rude here, I'm just curious of how busy can they be!

Sure thing. I was told on discord that infrastructure changes were going to be done before this could be looked though I don't know if that was with regards to do with priorities/timr or if those changes would help testing PRs like this one

Yea, @collinjackson and I talked offline and there is new testing infrastructure that would allow us to test this on CI, but this issue is blocking many people so I wanted to go ahead and submit it. We can currently test this locally, but we still need to set things up to test it on CI.