-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Invoice- Invoice report title is "user owes amount", but Report field shows Invoice YYYY-MM-DD #60274
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 @amyevans ( |
Triggered auto assignment to @mallenexpensify ( |
💬 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:
|
🚨 Edited by proposal-police: This proposal was edited at 2025-04-15 15:31:32 UTC. ProposalPlease re-state the problem that we are trying to solve in this issue.Invoice report title is "user owes amount", but Report field shows Invoice YYYY-MM-DD What is the root cause of that problem?In the ReportID field's App/src/components/ReportActionItem/MoneyRequestView.tsx Lines 805 to 808 in 31543a6
Instead, we should be creating the title using
What changes do you think we should make in order to solve the problem?In MonetyRequestView.tsx import then create the required title by using
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?N/A What alternative solutions did you explore? (Optional)N/A |
|
@mitarachim why can't you test on production? Please show what you see there next time. On production we don't render the Report field on the report since that's a new thing. It only shows on staging. As the contributor said it renders the parent report name. The parent report of the report prop in MoneyRequestView is the reportID shown in the URL On staging this is the reportName
On prod it's this
Since the backend data has the same name I think this appears as a new bug because we started displaying it on the frontend recently. The fundamental issue is the mismatch between NewDot ignoring the default report title formula and the backend using it. That's tracked by this issue. In the meantime maybe we can implement the contributors proposal to make the names match. Or we can revert, or consider it not a bug at this point. I'm not sure what's best. Removing the api deploy blocker label. |
Looks like it was introduced in #58597 - @waterim @luacmartins do y'all want to handle it as a follow up? |
Added to the follow up list |
@waterim Eep! 4 days overdue now. Issues have feelings too... |
@waterim Still overdue 6 days?! Let's take care of this! |
This is fixed in staging. |
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.28-1
Reproducible in staging?: Yes
Reproducible in production?: Unable to check
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: No, reproducible on hybrid only
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.3 / Chrome
App Component: Money Requests
Action Performed:
Precondition:
Expected Result:
Report field and invoice report title should be the same.
Actual Result:
Invoice report title is "user owes amount", but Report field shows Invoice YYYY-MM-DD
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6802647_1744720195508.20250415_202731.mp4
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @waterimThe text was updated successfully, but these errors were encountered: