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.34k stars 2.77k forks source link

[$250] Approvers - Expenses are added to the already submitted report before it is approved #47426

Open izarutskaya opened 1 month ago

izarutskaya commented 1 month 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!


Version Number: 9.0.20-0 Reproducible in staging?: Y Reproducible in production?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4857763 Email or phone of affected tester (no customers): fischer9966+081424aemp@gmail.com Logs: https://stackoverflow.com/c/expensify/questions/4856 Issue reported by: Applause-Internal team

Action Performed:

Precondition:

  1. Login as the admin and create Collect workspace
  2. Navigate to the collect workspace settings > More features > Enable workflows
  3. Navigate to workflows > Enable Approvals
  4. Enable delay submissions and set it to Manually
  5. Invite a new user with Gmail account as an employee

Steps:

  1. Login as the employee account in ND
  2. Navigate to the workspace chat
  3. Submit expense to the workspace for >100$
  4. Tap "Submit" button
  5. Submit another expense to the workspace

Expected Result:

A new expense is added to a new report, since the previous one has already been submitted

Actual Result:

A new expense is added to the already submitted report. Creation of a new report is possible only after approval of the previous one.

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

https://github.com/user-attachments/assets/4bf0166e-1c47-4e1c-942f-d6b6872d22fe

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021831342357487627692
  • Upwork Job ID: 1831342357487627692
  • Last Price Increase: 2024-09-18
Issue OwnerCurrent Issue Owner: @paultsimura
melvin-bot[bot] commented 1 month ago

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

izarutskaya commented 1 month ago

We think this issue might be related to the #collect project.

melvin-bot[bot] commented 1 month ago

@dylanexpensify Huh... This is 4 days overdue. Who can take care of this?

dylanexpensify commented 1 month ago

Reviewing today!

melvin-bot[bot] commented 3 weeks ago

@dylanexpensify Eep! 4 days overdue now. Issues have feelings too...

dylanexpensify commented 3 weeks ago

@Beamanator this was the issue we were chatting through in control, right? I think we have a solve here?

Beamanator commented 3 weeks ago

Hmmm honestly not sure if i agree, since this was report in Collect? But would be good to test if it happens in both!

It's still possible this has something to do with the recent "Instant submit for Control" changes

melvin-bot[bot] commented 3 weeks ago

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

melvin-bot[bot] commented 2 weeks ago

@dylanexpensify Huh... This is 4 days overdue. Who can take care of this?

melvin-bot[bot] commented 2 weeks ago

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

melvin-bot[bot] commented 2 weeks ago

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

melvin-bot[bot] commented 1 week ago

@paultsimura, @dylanexpensify Eep! 4 days overdue now. Issues have feelings too...

paultsimura commented 1 week ago

Bumped in Slack

trjExpensify commented 1 week ago

I'm pretty sure this came up somewhere in #wave-control recently with the changes to who can edit processing reports, I think @youssef-lr was working on it?

dylanexpensify commented 1 week ago

Ah yeah, agree - Advanced Approvals, we can move there!

paultsimura commented 1 week ago

I cannot reproduce this one 🤔

melvin-bot[bot] commented 1 week ago

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

melvin-bot[bot] commented 1 week ago

@paultsimura @dylanexpensify this issue is now 4 weeks old, please consider:

Thanks!

dylanexpensify commented 1 week ago

@izarutskaya can you reproduce?

melvin-bot[bot] commented 6 days ago

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

izarutskaya commented 6 days ago

I'll check and come back

izarutskaya commented 4 days ago

Not reproduced now Build v9.0.35-0

https://github.com/user-attachments/assets/8c8894cb-fdee-4202-b1aa-a4bf95517b7b

melvin-bot[bot] commented 2 days ago

@paultsimura, @dylanexpensify Still overdue 6 days?! Let's take care of this!

paultsimura commented 2 days ago

@dylanexpensify can we close based on https://github.com/Expensify/App/issues/47426#issuecomment-2351784240?

melvin-bot[bot] commented 1 day ago

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸