-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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] Expense - Error occurs when employee delete expense #48646
Comments
Triggered auto assignment to @kevinksullivan ( |
@kevinksullivan FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
ProposalPlease re-state the problem that we are trying to solve in this issue.Error is shown when an employee is trying to delete a canceled expense. What is the root cause of that problem?When the admin pays and cancels the expense, the expense report becomes in closed state. When the employee trying to delete the expense, the BE will return an expense. The error happens when we are trying to add another expense to the report too. So, it seems like the BE doesn't allow adding or deleting expenses to/from an expense report. What changes do you think we should make in order to solve the problem?Don't allow adding or deleting expenses to/from an expense report by returning false in Lines 1663 to 1665 in cc1c7d0
|
@kevinksullivan Eep! 4 days overdue now. Issues have feelings too... |
Job added to Upwork: https://www.upwork.com/jobs/~021834200864229937336 |
Looping in another BZ as I'm going OOO |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @akinwale ( |
📣 @akinwale 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
Triggered auto assignment to @sonialiap ( |
@akinwale, @sonialiap, @kevinksullivan Eep! 4 days overdue now. Issues have feelings too... |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@akinwale @sonialiap @kevinksullivan 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! |
@akinwale, @sonialiap, @kevinksullivan Still overdue 6 days?! Let's take care of this! |
We can move forward with @bernhardoj's proposal here. 🎀👀🎀 C+ reviewed. |
Triggered auto assignment to @mountiny, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
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: 9.0.29-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: N/A
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
The payment should be deleted without any problems
Actual Result:
An 'Unexpected error deleting the expenses' message occurs
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6592970_1725459918385.bandicam_2024-09-04_15-52-55-620.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @akinwaleThe text was updated successfully, but these errors were encountered: