-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[$250] Empty reports left for approval when all expenses are moved to different reports and Instant Submit is enabled #54787
Comments
Triggered auto assignment to @alexpensify ( |
This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989 |
Job added to Upwork: https://www.upwork.com/jobs/~021876502378955345300 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @rushatgabhane ( |
We are waiting for proposals here |
Btw, I think this is going to require some BE work. |
Yeah agreed. Btw, I've been able to reproduce this one, so I'm going to remove the label. Maybe we highlight this as a hotpick? |
waiting for proposals |
@alexpensify, @rushatgabhane Whoops! This issue is 2 days overdue. Let's get this updated quick! |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@alexpensify, @rushatgabhane Huh... This is 4 days overdue. Who can take care of this? |
@alexpensify @rushatgabhane this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
@alexpensify, @rushatgabhane Still overdue 6 days?! Let's take care of this! |
@waterim Did I see you comment you wanted to help on this one? 🙏 |
Hello, yes, Im Artem from Callstack and would like to help with this issue! |
Alright, I've assigned @waterim. Thanks for helping here! |
@alexpensify, @rushatgabhane, @waterim Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@waterim please keep us posted when you have an update. Thanks! |
@alexpensify, @rushatgabhane, @waterim Eep! 4 days overdue now. Issues have feelings too... |
Hello, Im working on this issue, was not able to fix this issue yet, but will have more time next week to finish! |
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:
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 @puneetlath
Slack conversation (hyperlinked to channel name): expense
Action Performed:
Expected Result:
When the last transaction is removed from a report, the report should be automatically deleted so long as there aren't any user comments on it. If there are user comments, the report should be changed to a
chat
report. This ensures that we maintain any relevant context and that approvers are not left with empty reports requiring approval.Actual Result:
The report remains in the approval queue, empty and requiring action from the approver.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @waterimThe text was updated successfully, but these errors were encountered: