Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.33k stars 2.76k forks source link

Report with submit + close and no approvals is in the processing state #48129

Closed kevinksullivan closed 19 hours ago

kevinksullivan commented 2 weeks ago

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


Reproducible in staging?: Yes Reproducible in production?: Yes 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: https://www.expensify.com/concierge/#/chat/13264620 Slack conversation: https://expensify.slack.com/archives/C041MSTR8J1/p1724729481646119?thread_ts=1724440636.400399&cid=C041MSTR8J1

Action Performed:

  1. Create workspace
  2. Configure delayed submission to be toggled off (default)
  3. Keep approvals toggled off (default)
  4. Assign expensify card to yourself (you can use expensifail.com or cardtest.expensify.com)
  5. Make a payment with the card from https://github.com/Expensify/Expensify/files/3974273/payButtons.html.zip
  6. Wait a day or two

Expected Result:

Report should move to closed state with no option to reimburse it, as it's a non-reimbursable expense.

Actual Result:

Platforms:

All platforms

Screenshots/Videos

image

View all open jobs on GitHub

melvin-bot[bot] commented 2 weeks ago

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

kevinksullivan commented 2 weeks ago

Since this has to do with the transactions/report state I figure it needs to be internal. CC @grgia

RachCHopkins commented 2 weeks ago

Internal issue created - @grgia I've assigned it to you as an interim measure.

RachCHopkins commented 1 week ago

Making weekly since the internal issue is a Daily.

RachCHopkins commented 4 days ago

Bumped Georgia on the Internal E/E issue. @kevinksullivan do I also need to keep this E/App issue open?

melvin-bot[bot] commented 3 days ago

@RachCHopkins 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!

RachCHopkins commented 19 hours ago

E/E is complete. We can close this one.