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

CRITICAL: [Track expense] Receipt leads to not here page when the scanning fails #38873

Closed
6 tasks done
kavimuru opened this issue Mar 23, 2024 · 15 comments
Closed
6 tasks done
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Internal Requires API changes or must be handled by Expensify staff

Comments

@kavimuru
Copy link

kavimuru commented Mar 23, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 1.4.56-0
Reproducible in staging?: y
Reproducible in production?: n (new feature)
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause interna team
Slack conversation:

Action Performed:

  1. Go to staging.new.expensify.com.
  2. Go to FAB > Track expense.
  3. Create a scan expense with invalid receipt.
  4. When the scanning fails, go to the expense details page.
  5. Click on the receipt.

Expected Result:

The receipt is rendered without issue.

Actual Result:

Not here page shows up when clicking on the receipt.
Note: To reproduce this bug reliably, please crearte new account

Workaround:

unknown

Platforms:

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

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6423496_1711131415018.20240323_021341.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01c7b85f133e64f453
  • Upwork Job ID: 1772330203765047296
  • Last Price Increase: 2024-03-25
Issue OwnerCurrent Issue Owner: @thienlnam
Issue OwnerCurrent Issue Owner: @thienlnam
@kavimuru kavimuru added DeployBlockerCash This issue or pull request should block deployment Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Mar 23, 2024
@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Mar 23, 2024
Copy link
Contributor

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

Copy link

melvin-bot bot commented Mar 23, 2024

Triggered auto assignment to @garrettmknight (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Hourly KSv2 labels Mar 23, 2024
Copy link

melvin-bot bot commented Mar 23, 2024

Triggered auto assignment to @johnmlee101 (Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

@kavimuru
Copy link
Author

@johnmlee101 FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

@shubham1206agra
Copy link
Contributor

@thienlnam This might be another BE issue.

@shubham1206agra
Copy link
Contributor

@garrettmknight This can be demoted to NAB as this feature is behind beta.

@garrettmknight garrettmknight removed the DeployBlockerCash This issue or pull request should block deployment label Mar 25, 2024
@garrettmknight
Copy link
Contributor

Flagged in vip-vsb

@garrettmknight garrettmknight added the Internal Requires API changes or must be handled by Expensify staff label Mar 25, 2024
Copy link

melvin-bot bot commented Mar 25, 2024

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

Copy link

melvin-bot bot commented Mar 25, 2024

Triggered auto assignment to Contributor Plus for review of internal employee PR - @ntdiary (Internal)

@garrettmknight
Copy link
Contributor

@thienlnam is gonna investigate.

@quinthar quinthar moved this from HIGH to CRITICAL in [#whatsnext] #vip-vsb Mar 26, 2024
@quinthar quinthar changed the title Track expense - Receipt leads to not here page when the scanning fails CRITICAL: Track expense - Receipt leads to not here page when the scanning fails Mar 26, 2024
@quinthar quinthar changed the title CRITICAL: Track expense - Receipt leads to not here page when the scanning fails CRITICAL: [Track expense] Receipt leads to not here page when the scanning fails Mar 30, 2024
@melvin-bot melvin-bot bot added the Overdue label Mar 30, 2024
@garrettmknight
Copy link
Contributor

@thienlnam any update on this one?

@shubham1206agra
Copy link
Contributor

@garrettmknight Can you retest this?

@thienlnam
Copy link
Contributor

Ah yeah, I was looking into this and I think I fixed this as well when working on the edit tracked expense flow

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Apr 3, 2024
Copy link

melvin-bot bot commented Apr 6, 2024

@garrettmknight @ntdiary @thienlnam this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

Copy link

melvin-bot bot commented Apr 8, 2024

@garrettmknight, @ntdiary, @thienlnam Huh... This is 4 days overdue. Who can take care of this?

@melvin-bot melvin-bot bot removed the Overdue label Apr 9, 2024
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 Engineering Internal Requires API changes or must be handled by Expensify staff
Projects
No open projects
Status: CRITICAL
Development

No branches or pull requests

6 participants