-
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
[HOLD for payment 2023-10-30] [HOLD for payment 2023-09-15] [$1000] HIGH [W4 Scan Receipts] LHN shows user paid $0.00
when sending first money request via Scan
#25778
Comments
Triggered auto assignment to @maddylewis ( |
Bug0 Triage Checklist (Main S/O)
|
user paid $0.00
when sending first money request via Scanuser paid $0.00
when sending first money request via Scan
Job added to Upwork: https://www.upwork.com/jobs/~015c1a629214ef23ce |
Current assignee @maddylewis is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @thesahindia ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.LHN shows user paid $0.00 when sending first money request via Scan What is the root cause of that problem?
Lines 1208 to 1225 in f0eeac6
What changes do you think we should make in order to solve the problem?Add the below code to show the correct status.
The above will show the message if any receipt in the Report is being scanned. To show only for the last message, we can only check with the IOUReceipt_trimmed.mp4Though it's not main issue listed here, i'd like to mention that if such message has to be shown somehwere else, like in the report where the IOU was created, similar logic can be used in functions like |
@ygshbht proposal looks mostly good to me, but what is the first undefined param here |
📣 @thesahindia Please request via NewDot manual requests for the Reviewer role ($1000) |
📣 @ygshbht You have been assigned to this job! |
We can sort that out in the PR review. Assigning you to the issue. |
@luacmartins Thanks. I'll have the PR ready within a day. The
|
@luacmartins @thesahindia The PR is ready for review! |
I'll be taking this one over as requested by @thesahindia. |
📣 @allroundexperts Please request via NewDot manual requests for the Contributor role ($1000) |
Can you guys help look into this please? I'm going to track this now in wave7. CC: @youssef-lr |
@trjExpensify I think that was the inital job but had to revert due to change in requirements. You can follow the dicussion in the PR or read only from here. Unfortunately, there might have been confusions |
The discussion seems to point out the expectation to me. Especially as you go further down to say here. Why did we revert it? It shouldn't be reading |
@trjExpensify Sorry, I don't understand. |
Right, and prior to the receipt scan completing then the title of the expenseReport (providing this is the only request on the report) is |
That's what you see, don't you? #25906 (comment) |
Ah, so that's the diff. I'm not saying that. For the expenseReport... Right now the LHN title is |
So instead of |
Correct, owes, and more specifically given we have two types of reports in question here: iouReport expenseReport |
It's the expenseReport one. If I request money from you, the expense goes on a report of type |
Ok, so in iou, |
That's correct!
I don't understand the latter as an option because this PR has been deployed already. I also don't think this needs a new issue, so can we please create a follow-up PR linked to this issue to fix it? |
@trjExpensify Created PR |
Based on my calculations, the pull request did not get merged within 3 working days of assignment. Please, check out my computations here:
On to the next one 🚀 |
user paid $0.00
when sending first money request via Scanuser paid $0.00
when sending first money request via Scan
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.88-11 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 2023-10-30. 🎊 After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.
For reference, here are some details about the assignees on this issue:
As a reminder, here are the bonuses/penalties that should be applied for any External issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Okay, so with that now taken care of I think we can proceed to close this out for good now. Thanks everyone! |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Scan
<user> paid $0.00
Expected Result:
The LHN should show
Scan in progress
Actual Result:
LHN says
<user> paid $0.00
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: