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

Empty report submitted and appeared in LHN with GBR #54224

Open
2 of 8 tasks
m-natarajan opened this issue Dec 17, 2024 · 9 comments
Open
2 of 8 tasks

Empty report submitted and appeared in LHN with GBR #54224

m-natarajan opened this issue Dec 17, 2024 · 9 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Needs Reproduction Reproducible steps needed Overdue

Comments

@m-natarajan
Copy link

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:
Reproducible in staging?: Needs Reproduction
Reproducible in production?: Needs Reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @quinthar
Slack conversation (hyperlinked to channel name): expense

Action Performed:

  1. Log in and observe LHN

Expected Result:

Reports should not be allowed to be submitted if they are empty.

Actual Result:

An empty report was successfully submitted by user and appeared in the LHN for approval with GBR

Workaround:

Unknown

Platforms:

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

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

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub

@m-natarajan m-natarajan added Daily KSv2 Needs Reproduction Reproducible steps needed Bug Something is broken. Auto assigns a BugZero manager. labels Dec 17, 2024
Copy link

melvin-bot bot commented Dec 17, 2024

Triggered auto assignment to @MitchExpensify (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.

@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@MitchExpensify
Copy link
Contributor

It seems weird that the LHN row shows $10, but the report shows $0. @OfstadC do you recall the steps you took to submit this report? Do you see it showing as $0?

@garrettmknight garrettmknight moved this to Bugs and Follow Up Issues in [#whatsnext] #expense Dec 17, 2024
@OfstadC
Copy link
Contributor

OfstadC commented Dec 17, 2024

@MitchExpensify I don't remember 😬

This is what I see
image
image

I think I deleted something around that time? There was a RyanAir email (lingering spam from booking Expensicon flights for customers) that was added around the same time I deleted. Maybe that's tied to this? 🤔
image

image

I don't remember exact steps and I didn't notice the $0 report. But if it's tied to the Ryanair piece - I deleted that expense in Classic I think.

@MitchExpensify
Copy link
Contributor

That's great context @OfstadC , was this an Expensify Travel booking?

@OfstadC
Copy link
Contributor

OfstadC commented Dec 18, 2024

The original Ryanair booking? It was from back when we booked travel manually for ExpensiCon guests. So I booked directly with Ryanair with my Expensify Card. I imagine the follow up emails and subsequent ads are being added as Travel documents/receipts (if there is an amount on them)? 🤔

@MitchExpensify
Copy link
Contributor

I'm tempted to chalk this up to old travel booking weirdness and close it. Essentially it looks like a Ryanair promotional email was forwarded to receipts@, no amount was picked up by SmartScan, and that expense was added to a Trip. That trip then showed in @quinthar's account in an empty state.

@melvin-bot melvin-bot bot added the Overdue label Dec 23, 2024
Copy link

melvin-bot bot commented Dec 23, 2024

@MitchExpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!

@MitchExpensify
Copy link
Contributor

Coming back on this, it seems more likely related to the trip expense being deleted in OldDot. Can we even create trips like this anymore to try to reproduce @OfstadC ? My theory is that this happened because the deleted expense was on a trip and we don't delete the trip even though it is in an empty state

Possible steps to reproduce:

  1. Create a trip
  2. Submit an expense for the trip with a receipt
  3. As the approver, confirm in NewDot that the trip and expense exist (i.e. Are not in an empty state)
  4. As the submitter, delete the expense in OldDot
  5. As the approver, confirm in NewDot if the expense exists in an empty state or not

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Dec 24, 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 Needs Reproduction Reproducible steps needed Overdue
Projects
Status: Bugs and Follow Up Issues
Development

No branches or pull requests

4 participants