-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[Due for payment 2025-06-02] [Due for payment 2025-05-30] [Report Creation UI] [$250] Expense - Not here page opens when admin clicks Move expense in member's report #62575
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 @slafortune ( |
Triggered auto assignment to @grgia ( |
💬 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:
|
Likely #62270 |
Job added to Upwork: https://www.upwork.com/jobs/~021925612414512804213 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @Pujan92 ( |
I reproduced in staging v9.1.49-6 and not in prod v9.1.49-0 |
Yeah, @luacmartins @Pujan92 and I are looking into it now. |
I can repro in the 2025-05-22_20-04-47.mp4It's gotta' be something to do with the admin/approver permissions to move the expense. |
@twilight2294 all yours. Please work on a PR |
In the video, employee has 2 reports, and admin has 1 report. If we open one of the employee reports then the reports list will display only the admin report, which is obviously incorrect. So we need to fix it here |
sure, thanks!, working on it ASAP |
@luacmartins Could you please take a look at #62575 (comment), removing change report will only hide the bug for specific case and not fixing the actual bug here. |
We're not removing the option to change report. We're updating the flow to use the correct accountID and also only show the field as editable if there are available reports |
Line 9687 in 0d7f742
Isn't this( |
PR ready for review |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.50-0 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2025-05-30. 🎊 For reference, here are some details about the assignees on this issue:
|
@grgia @slafortune @jnowakow / @twilight2294 The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button] |
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.51-6 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2025-06-02. 🎊 For reference, here are some details about the assignees on this issue:
|
@grgia @slafortune @jnowakow / @twilight2294 The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button] |
Uh oh!
There was an error while loading. Please reload this page.
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.49-5
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail: Exp
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: Mac 15.4 / Chrome
App Component: Money Requests
Action Performed:
Precondition:
Expected Result:
Report list will open with the current report selected.
Actual Result:
Not here page opens.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6839124_1747913566720.20250522_192759.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @slafortuneThe text was updated successfully, but these errors were encountered: