-
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
[HOLD for payment 2025-01-22] [$250] LHN - RBR is missing from the report when it fails to export to QBO #53924
Comments
Triggered auto assignment to @joekaufmanexpensify ( |
Reproduced |
Job added to Upwork: https://www.upwork.com/jobs/~021866980041596226997 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @thesahindia ( |
cc @arosiclair as I see you originally implemented this |
I'll manage this. I might take it internal if I get the cycles. |
TY! |
@arosiclair, @joekaufmanexpensify, @thesahindia Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
I cannot trigger the error "failed to export the report to QuickBooks Online. @joekaufmanexpensify Could you give me a trick to reproduce it, please? Thanks |
@truph01 did you follow the steps in OP of the issue? I had no issue reproducing it with those steps. |
@arosiclair, @joekaufmanexpensify, @thesahindia Huh... This is 4 days overdue. Who can take care of this? |
@joekaufmanexpensify Yes, I followed the steps and got stuck on the screen below, where the message remains as "started exporting this report to QuickBooks Online..." instead of updating to "failed to export this report..." as mentioned by the OP: |
Interesting. Even after refreshing the browser? |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Yes |
@arosiclair, @joekaufmanexpensify, @thesahindia 6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
Got it. I'll try reproducing again to make sure I still can. |
I just tried again and could still reproduce it. @truph01 after exporting, try syncing the QBO connection to see if that triggers the error on the report. |
Moving this to weekly for now as I am OOO after today through EOY. If this needs any BZ urgently before then, please ask in Slack. Otherwise, I will continue to assist when I return. |
📣 @DylanDylann 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
📣 @DylanDylann 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
@DylanDylann @arosiclair @IuliiaHerets @joekaufmanexpensify I just opened a PR with the fix for this issue 😄 |
Today I dedicated my time to write unit tests for this PR but I haven't finished them yet. |
PR was merged! 🚀 Thanks @DylanDylann and @arosiclair 😄 |
Wooo, TY! |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.85-4 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-01-22. 🎊 For reference, here are some details about the assignees on this issue:
|
@DylanDylann @joekaufmanexpensify @DylanDylann 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] |
@DylanDylann could you handle checklist here? TY! |
BugZero Checklist:
Bug classificationSource of bug:
Where bug was reported:
Who reported the bug:
Regression Test ProposalPrecondition:Test:
Do we agree 👍 or 👎 |
No new test needed. I checked in test rail and this case is already covered by https://expensify.testrail.io/index.php?/cases/view/3284910&group_by=cases:section_id&group_order=asc&display_deleted_cases=0&group_id=309135 |
All set for payment! Only payment here is $250 to @DylanDylann for C+ review via upwork. |
@DylanDylann $250 sent and contract ended! |
Upwork job closed. |
All set, thanks everyone! |
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.74-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5324684&group_by=cases:section_id&group_id=309135&group_order=asc
Issue reported by: Applause Internal Team
Action Performed:
Expected Result:
RBR should be visible.
Actual Result:
RBR is missing from the report when it fails to export to QBO.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6690691_1733905188166.bandicam_2024-12-11_09-04-56-770.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @joekaufmanexpensifyThe text was updated successfully, but these errors were encountered: