-
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-11-01] [HOLD for payment 2023-10-31] Fix the double message on the iou report when we pay a money request using wallet #30075
Comments
Triggered auto assignment to @laurenreidexpensify ( |
Bug0 Triage Checklist (Main S/O)
|
@techievivek you're putting a PR up for this yeah? |
Yes, I have added the PR, I have a quick question for @situchan |
Perfect! |
This is the updated PR for fixing the settled message issues that we saw in the original GH: #30112 |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.89-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 2023-10-31. 🎊 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:
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.90-2 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-11-01. 🎊 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:
|
Checklist time! Bump, @situchan. |
Same, awaiting the checklist from @situchan Melv. |
@trjExpensify, @techievivek, @situchan Whoops! This issue is 2 days overdue. Let's get this updated quick! |
As wallet feature is in beta, yet to add regression test case. |
So payment here is $500 to @situchan for the C+ review of the internal PR. |
Q on the regression for wallet on another issue. |
Sent you an offer :) |
Paid! |
Coming from : https://expensify.slack.com/archives/C01GTK53T8Q/p1697796531033319?thread_ts=1697788964.150409&cid=C01GTK53T8Q
Action Performed:
Pre-requisite
Expected Result:
We should see a single message that the user needs to update their wallet.
Actual Result:
We're seeing two system messages asking the user to update their wallet and bank account.
CC @trjExpensify @kevinksullivan
The text was updated successfully, but these errors were encountered: