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

IOS - Deep link doesn't direct to the IOS app when creating a new account #13882

Closed
1 task done
kbecciv opened this issue Dec 29, 2022 · 4 comments
Closed
1 task done
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2

Comments

@kbecciv
Copy link

kbecciv commented Dec 29, 2022

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Action Performed:

  1. Open the app
  2. In the email field enter a email the new Expensify account and click on continue
  3. Verify there’s a message saying that a Magic Link has been sent to the email address
  4. Copy the dip link and paste it into notepad
  5. In the notebook, change to staging
  6. Click this dip link

Expected Result:

The deep link should point to the iOS app when creating a new account

Actual Result:

Deep link doesn't direct to the IOS app when created a new account

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • iOS / native

Version Number: 1.2.45.0

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers):

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Bug5879830_dip_link_ios_2812.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

@kbecciv kbecciv added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Dec 29, 2022
@melvin-bot melvin-bot bot locked and limited conversation to collaborators Dec 29, 2022
@jasperhuangg jasperhuangg self-assigned this Jan 4, 2023
@jasperhuangg
Copy link
Contributor

This is actually expected, if we want to deeplink to the native app then the URL needs to have a different format, it's actually correct that we redirect to the web browser for any links that look like https://staging.new.expensify.com...

I don't think this is a bug per se. I think in other apps when you try to deeplink it will also bring you to the web app to do validation. I'm electing to close this out

@kbecciv kbecciv changed the title IOS - Dip link doesn't direct to the IOS app when creating a new account IOS - Deep link doesn't direct to the IOS app when creating a new account Feb 15, 2023
@marcaaron
Copy link
Contributor

@jasperhuangg I'm not sure if I understood the decision. Why would a link with staging.new.expensify.com not be an acceptable deep link? We have the acceptable schemes listed here:

prefixes: [
'new-expensify://',
'https://www.expensify.cash',
'https://staging.expensify.cash',
'http://localhost',
CONST.NEW_EXPENSIFY_URL,
CONST.STAGING_NEW_EXPENSIFY_URL,
],

@MelvinBot
Copy link

Looks like something related to react-navigation may have been mentioned in this issue discussion.

As a reminder, please make sure that all proposals are not workarounds and that any and all attempt to fix the issue holistically have been made before proceeding with a solution. Proposals to change our DeprecatedCustomActions.js files should not be accepted.

Feel free to drop a note in #expensify-open-source with any questions.

@jasperhuangg
Copy link
Contributor

@marcaaron good shout, didn't realize that we also handled links with the format staging.new.expensify.com to the iOS app. This issue was correctly reported.

I'm gonna keep this closed since there's a dupe issue already: #15158

@Expensify Expensify unlocked this conversation Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2
Projects
None yet
Development

No branches or pull requests

5 participants