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

Improve user experience #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

duncanhsieh
Copy link

Assume that the browser triggers the visibilityChange event when the client successfully opens deeplink, this script can cancel the opening of the app store.

Assume that the browser triggers the visibilityChange event when the user successfully opens deeplink.
When the user successfully opens deeplink, this script can cancel the opening of the app store.
@mderazon
Copy link
Owner

mderazon commented Jun 1, 2020

Thanks, i'll have a look at this a bit later today

@mderazon
Copy link
Owner

mderazon commented Jun 1, 2020

Hey @duncanhsieh, what was your experience before you added this ?
If the app was installed, and the deeplink was opened successfully would the app store still open behind ?
How did it behave ?

@duncanhsieh
Copy link
Author

Hey @duncanhsieh, what was your experience before you added this ?
If the app was installed, and the deeplink was opened successfully would the app store still open behind ?
How did it behave ?

Hey, I make a video for this issue.
https://imgur.com/a/00iUgWk
When deeplink is opened, the appstore immediately covers the app in iphone.

@mderazon
Copy link
Owner

mderazon commented Jun 2, 2020

When deeplink is opened, the appstore immediately covers the app in iphone.

Thanks for the video, don't have an ios device with me, i'll try to reproduce this on the simulator but it looks similar to #9, which is why I recommended using universal links. If this indeed makes it a smoother experience then great

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants