braze-inc / braze-cordova-sdk

Public repo for the Braze Cordova SDK
https://www.braze.com
Other
21 stars 62 forks source link

iOS - Push notification when the app is open #45

Closed asangadev closed 3 years ago

asangadev commented 4 years ago

Hi @radixdev,

Is it possible to receive a push notification while the app is running and open? Please advise.

radixdev commented 4 years ago

Hi @asangadev ,

Have you tried routing this through support@braze.com? This seems better suited to that avenue, unless you're describing a bug.

asangadev commented 4 years ago

@radixdev, cheers! they got back to me with below:

https://www.braze.com/docs/developer_guide/platform_integration_guides/ios/push_notifications/integration/#step-5-enable-push-handling

asangadev commented 4 years ago

Hi @radixdev,

I know this is not a bug report. This is a feature request from us. Do you think you can show us how to implement below with current plugin for iOS?

Screen Shot 2019-10-03 at 11 04 11 am
radixdev commented 4 years ago

Hi @asangadev ,

We're adding this to our Cordova product roadmap. Thank you for bringing this to our attention.

asangadev commented 4 years ago

Cheers!

asangadev commented 4 years ago

@Bucimis ,

Do you guys have an update on this or any instructions how to enable this for iOS users? Please advise.

asangadev commented 4 years ago

@Bucimis

Any update?

Bucimis commented 4 years ago

@asangadev we're aiming to have this in 4 weeks-ish

xueligoh commented 4 years ago

Hihi, we are also not receiving the push notification when the app is running in the foreground for android. Possible to check if this is also not a feature available for android currently? And if so, whether it is also included in the roadmap for android? Thanks in advanced!

(Edited): Noticed that notifications sent out via braze will be available in my notification drawer but not notifications from firebase console when the app is opened. Think that I might be looking at a different issue.

(Edited again) I think our issues got to do with this instead. Will raise separately if necessary. Thanks!

asangadev commented 3 years ago

@xueligoh Actually I never had that foreground issue with Android. It's working for me on Android.

Bucimis commented 3 years ago

@xueligoh +1 to @asangadev, this should already work on Android

asangadev commented 3 years ago

@Bucimis any progress update here? We would love to add this feature to our app soon.

This feature is a must have for our iOS users.

Please keep us in the loop.

Bucimis commented 3 years ago

@asangadev we're expecting to have support for this in the next 4-6 weeks

asangadev commented 3 years ago

@Bucimis @radixdev

Thanks guys! This is working now with: https://github.com/Appboy/appboy-cordova-sdk/blob/master/CHANGELOG.md#2160