Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DeepLinking Documentation Needed For React Native #5025

Open
vikas5914 opened this issue Apr 11, 2019 · 11 comments
Open

DeepLinking Documentation Needed For React Native #5025

vikas5914 opened this issue Apr 11, 2019 · 11 comments
Labels
amplify/js Issues tied to JS feature-request Request a new feature transferred This issue was transferred from another Amplify project

Comments

@vikas5914
Copy link

vikas5914 commented Apr 11, 2019

Documentation needed for DeepLinking
A clear and concise documentation is needed in order to implement fully functioning DeepLinking through Pinpoint Notification.

Steps Required:

  1. Integration on Android
  2. Integration on iOS
  3. Integration on React-Native App
@powerful23
Copy link
Contributor

@vikas5914 hi, can you provide more about fully functioning DeepLinking through Pinpoint Notification.? For using pinpoint notification, the doc is here: https://aws-amplify.github.io/docs/js/push-notifications

@vikas5914
Copy link
Author

@powerful23

Hi,

For fully functioning DeepLinking through Pinpoint Notification, i have mentioned steps that is required to fully implement the DeepLinking through Pinpoint in React Native.

and for the doc available on the https://aws-amplify.github.io/docs/js/push-notifications doesn't have DeepLinking inside.

@powerful23
Copy link
Contributor

@vikas5914 I see. Since both Open an url and Open a deep link are not implemented yet, I will mark this as feature request. As for now you have to implement this by yourself.

@powerful23 powerful23 added the feature-request Request a new feature label Apr 16, 2019
@stale
Copy link

stale bot commented Jun 15, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@miguelocarvajal
Copy link

@powerful23 What would be the way to implement this directly?

@danilojpferreira
Copy link

UP

@thenderson55
Copy link

It doesn't actually say how to implement it.

https://docs.aws.amazon.com/cli/latest/reference/pinpoint/send-messages.html

Action -> (string)

The action to occur if the recipient taps the push notification. Valid values are:

OPEN_APP - Your app opens or it becomes the foreground app if it was sent to the background. This is the default action.
DEEP_LINK - Your app opens and displays a designated user interface in the app. This action uses the deep-linking features of the Android platform.
URL - The default mobile browser on the recipient's device opens and loads the web page at a URL that you specify.

@spmatthews03
Copy link

What is the status of this? this is a pretty crucial defect for some projects.

@the-smart-home-maker
Copy link

Since both Open an url and Open a deep link are not implemented yet, I will mark this as feature request. As for now you have to implement this by yourself.

@powerful23 so do I understand it right, that this is still not yet implemented? I was trying to figure out for hours why deep linking from a pinpoint notification in React Native is not working as expected but now I understand why it does not work ;-)

@vikas5914
Copy link
Author

Since both Open an URL and Open a deep link are not implemented yet, I will mark this as feature request. As for now you have to implement this by yourself.

@powerful23 so do I understand it right, that this is still not yet implemented? I was trying to figure out for hours why deep linking from a pinpoint notification in React Native is not working as expected but now I understand why it does not work ;-)

The way we implement this two years ago was to send a screen name in notification data and handle the navigation by ourselves. it works fine for years.

@the-smart-home-maker
Copy link

Ok thank you - I will implement it myself as well

@stocaaro stocaaro added the transferred This issue was transferred from another Amplify project label Jan 17, 2023
@stocaaro stocaaro transferred this issue from aws-amplify/amplify-js Jan 17, 2023
@timngyn timngyn added the amplify/js Issues tied to JS label Feb 10, 2023
@abdallahshaban557 abdallahshaban557 added the p4 guide needed/net new info label Mar 25, 2023
@nadetastic nadetastic removed the p4 guide needed/net new info label Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
amplify/js Issues tied to JS feature-request Request a new feature transferred This issue was transferred from another Amplify project
Projects
None yet
Development

No branches or pull requests