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

Close reports for the combo of Instant Submit + No Approvals + No Payments #49131

Open
garrettmknight opened this issue Sep 12, 2024 · 5 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Reviewing Has a PR in review Weekly KSv2

Comments

@garrettmknight
Copy link
Contributor

garrettmknight commented Sep 12, 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: N/A
Reproducible in staging?: Y
Reproducible in production?: Y
Issue reported by: @garrettmknight
Slack conversation: https://expensify.slack.com/archives/C06ML6X0W9L/p1726136887331749?thread_ts=1725892977.799399&cid=C06ML6X0W9L

Action Performed:

  1. Sign up with a new account
  2. Select 'Manage my team's expenses'
  3. Enter a first name
  4. Enable Workflows in the workspace
  5. Navigate to the Workflows tab
  6. Disable Payments
  7. Navigate to your workspace chat
  8. Submit a manual expense

Expected Result:

Since there is no approval or payment action for anyone to take, the report should be Closed. This will result in each individual expense in this configuration creating a closed report.

Actual Result:

The expense is added to a report in the Processing state with inaccurate next steps: Waiting for [workspace admin] to pay expense(s).

Workaround:

No workaround.

Platform:

All

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub

Issue OwnerCurrent Issue Owner: @srikarparsi
@garrettmknight garrettmknight added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Sep 12, 2024
@garrettmknight garrettmknight self-assigned this Sep 12, 2024
Copy link

melvin-bot bot commented Sep 12, 2024

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

@trjExpensify
Copy link
Contributor

Because of a couple of other bugs atm we're working on elsewhere, you might want to switch to OldDot settings to get your workspace into the correct state to develop with @srikarparsi, or basically these are the three important properties on the workspace for the right condition:

  • scheduledSubmit=instantly
  • approvals = submit & close (can't remember what this one is technically, APPROVAL_MANUAL maybe 😅)
  • reimbursement=none

@srikarparsi
Copy link
Contributor

srikarparsi commented Sep 13, 2024

Awesome, I was able to replicate this pretty easily using @garrettmknight's steps in the issue body, didn't have to do anything in Old Dot. We just don't want the "Waiting for Srikar to pay expense(s)" because payments is not enabled right.

image

@trjExpensify
Copy link
Contributor

Awesome, I was able to replicate this pretty easily using @garrettmknight's steps in the issue body

Ah yeah, I mean it's impossible in NewDot to get the workspace into that state because of the bugs. So if you need to to test what happens to the report state when all of those three are set as such, you can get the config there via OldDot.

We just don't want the "Waiting for Srikar to pay expense(s)" because payments is not enabled right.

I think you've figured out since that the nextSteps text isn't the only scope of what's wrong with this issue. The underlying report is in the processing state instead of closed when "instant + no approvals + no payments" is configured.

@shubham1206agra
Copy link
Contributor

@garrettmknight Can you assign me this issue as reviewer?

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. Reviewing Has a PR in review Weekly KSv2
Projects
Status: Release 2.5: SuiteWorld (Sept 9th)
Development

No branches or pull requests

5 participants