-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Expense - Unable to move expense to the report of a track expense. #62154
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
Comments
Triggered auto assignment to @greg-schroeder ( |
Triggered auto assignment to @robertjchen ( |
💬 A slack conversation has been started in #expensify-open-source |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
Triggered auto assignment to @pecanoro ( |
💬 A slack conversation has been started in #expensify-open-source |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
Actually there's API error so I think we have to modify ![]() Actually this happens also when change is initialised from report field so I don't think it was caused by #61839 Screen.Recording.2025-05-16.at.11.32.33.mov |
Got it, thanks for the additional info! |
Found the logs for that requestid
I guess this is expected since that check has been there for 9 months now, we're probably just not showing an error on the frontend? @mitarachim Could you please confirm if the the expense was being moved onto an non-reimbursable report with instant submit enabled and no approvers in your test case? 🙏 |
Hello @robertjchen sorry for very late respond. 2025-05-18.9.34.23.AM.mp4 |
Got it, thanks for the confirmation- since this appears to be more of a backend issue not related to the recent frontend PR, I'll remove the |
I tried reproducing this but I encountered a different error. I see @youssef-lr do you have more context on why this check was added? |
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.1.46-0
Reproducible in staging?: Yes
Reproducible in production?: No
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
Device used: Motorola MotoG60 - Android 12 - Chrome / Windows 10 - Chrome
App Component: Money Requests
Action Performed:
Prerequisite: Account has a Self DM and at least one workspace available.
Prerequisite 2: Delayed submissions and approvals disabled.
Expected Result:
User should be able to move an expense to a different report without any issue.
Actual Result:
When moving an expense to the report of a different expense, the user lands on the report of the second one, but no changes can be seen on amount or information. After that, when tapping on back button, the user lands on the report of the moved expense, but all the information fields are empty, and when returning to chat, both expenses remain separated on different reports.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6832595_1747350753617.Moved.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: