React-native-onesignal: How to disable the "Open Website?" alert, when payload includes launchURL

[Salar-g]

Description:
I have a launchURL in my notification payload. LaunchURL is used for navigating to a specific page using deep linking. However every time a notification is received, an Alert also pops-up that says "Open Website? Would you like to open myApp:main..". The deep-linking works as expect. I wonder if there is was prevent this Alert to pop-up.

(your description here)

Environment
"react-native-onesignal": "^3.6.4",
"react-native": "0.61.2",

Steps to Reproduce Issue:

1. Install the OneSignal SDK using npm into your project
2. add deep linking
3. send a notification with launchURL
4. the below notification will be shown

Related: #1016

[rgomezp]

Duplicate of https://github.com/OneSignal/react-native-onesignal/issues/1140

[dickfickling]

@rgomezp you closed this as a duplicate of #1140 (which is... this ticket). Did you mean it's a dupe of OneSignal/OneSignal-iOS-SDK#777? just want to make sure the work to fix it is being tracked.

[rgomezp]

Yes. It was a mistake. Thanks for pointing that out.

[stu-dev]

Hi @rgomezp, I don't think this is technically a duplicate of the iOS issue #777.

The iOS issue wants to disable the "Open Website" alert so that the deeplink it could be handled natively, whereas on the React Native side I'm assuming most of us just want it to automatically open the deeplink without the alert? It seems like 2 different expected behaviours.

[leethree]

Yes I just disabled the iOS prompt now deep linking no longer works automatically.

I tried to manually handle launchURL in setNotificationOpenedHandler but it doesn't seem to work on cold launch from notification.

[tomwhale]

Still seeing this issue! The alerts have gone, but deep linking isn't working. If you have any hints on how to fix it @rgomezp I am happy to do a PR.

Thanks!

[rgomezp]

Howdy everyone,
We did resolve the native SDK issue 777 in the latest release which updates the underlying native iOS SDK to the version with the fix.

@stu-dev can you please upgrade and try that release to see if the issue is resolved?

[tomwhale]

@rgomezp I can confirm the Alert no longer shows, but deep linking does not work.

[stu-dev]

Hi @rgomezp to echo what @tomwhale said - on upgrade the message disappears but now deep linking does not work properly

[rgomezp]

Thanks everyone for the info.

@gonzalonarbaiz is investigating.

To confirm, this deep linking issue was not occurring for you prior to the alert issue correct?

[tomwhale]

Hi,

It was happening before. From the app closed state, the deep linking did not work.

Something was released which caused the alert prompt when deeplinking. Deeplinking worked, but we had the alert which was not ideal.

Then another fix went out which fixed the alert, but deep linking no longer works again.

[azai91]

Any update on this?

[rgomezp]

Related: #1016

[rgomezp]

Howdy,
Since the alert is resolved, I will close this issue.

The deeplinking issue is covered by #1016 . Please continue tracking the issue there.