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

[$250] Chat - Unable to redirect to hybrid app via link #56521

Open
2 of 8 tasks
izarutskaya opened this issue Feb 7, 2025 · 13 comments
Open
2 of 8 tasks

[$250] Chat - Unable to redirect to hybrid app via link #56521

izarutskaya opened this issue Feb 7, 2025 · 13 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors

Comments

@izarutskaya
Copy link

izarutskaya commented Feb 7, 2025

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


Version Number: v9.0.95-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: No, reproducible on hybrid only
If this was caught during regression testing, add the test name, ID and link from TestRail: #56228
Email or phone of affected tester (no customers): N/A
Issue reported by: Applause Internal Team
Device used: iPhone 14/ iOS 17.6.1
App Component: Other

Action Performed:

  1. Go to https://staging.new.expensify.com/
  2. On Account A, navigate to any report (e.g., a concierge, chat, or expense report) and note the report ID.
  3. Log out of Account A.
  4. Open the report URL using the noted report ID (e.g., https://staging.new.expensify.com/r/72105063) while logged out.

Expected Result:

User redirects to hybrid app via link

Actual Result:

Unable to redirect to hybrid app via link

Workaround:

Unknown

Platforms:

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6736005_1738916503622.ScreenRecording_02-07-2025_08-42-13_1.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021889296012784339930
  • Upwork Job ID: 1889296012784339930
  • Last Price Increase: 2025-02-11
Issue OwnerCurrent Issue Owner: @shubham1206agra
@izarutskaya izarutskaya added Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 labels Feb 7, 2025
Copy link

melvin-bot bot commented Feb 7, 2025

Triggered auto assignment to @garrettmknight (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@rushatgabhane
Copy link
Member

rushatgabhane commented Feb 7, 2025

@izarutskaya is this issue occuring on production too?

i.e.

  1. uninstall staging app
  2. install production app,
  3. and click on production link

@rushatgabhane
Copy link
Member

@garrettmknight could you please add deploy blocker label until we know for sure if it is on production

@OmarKoueifi
Copy link
Contributor

How do I install production app

@rushatgabhane
Copy link
Member

From play/app store

@OmarKoueifi
Copy link
Contributor

😅 Obviously! I just saw a different icon, so I thought it was an older app or something else

@OmarKoueifi
Copy link
Contributor

@izarutskaya, @rushatgabhane, Based on my code changes, I don't believe this issue is related to my PR. My changes only introduced a condition that handles cases where a report is not accessible for a user who just signed in.

To confirm, could you test with a link to a report that the user has access to (e.g., a public room or Concierge chat while signed in)? If the issue still occurs in that case, then it’s likely unrelated to my PR.

I also tested this on the production app, and the link opened in Safari. Let me know if there's anything else I should check!

@rushatgabhane
Copy link
Member

perfect! thanks for confirming 😃

@melvin-bot melvin-bot bot added the Overdue label Feb 10, 2025
Copy link

melvin-bot bot commented Feb 10, 2025

@garrettmknight Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@garrettmknight garrettmknight added the External Added to denote the issue can be worked on by a contributor label Feb 11, 2025
@melvin-bot melvin-bot bot changed the title Chat - Unable to redirect to hybrid app via link [$250] Chat - Unable to redirect to hybrid app via link Feb 11, 2025
Copy link

melvin-bot bot commented Feb 11, 2025

Job added to Upwork: https://www.upwork.com/jobs/~021889296012784339930

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Feb 11, 2025
Copy link

melvin-bot bot commented Feb 11, 2025

Triggered auto assignment to Contributor-plus team member for initial proposal review - @shubham1206agra (External)

@melvin-bot melvin-bot bot removed the Overdue label Feb 11, 2025
@shubham1206agra
Copy link
Contributor

@war-in
Copy link
Contributor

war-in commented Feb 11, 2025

Hi, yes, @shubham1206agra is right
We're working on NewDot deeplinks support in HybridApp in the mentioned PR

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 External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors
Projects
None yet
Development

No branches or pull requests

6 participants