-
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
'Settled' and 'cancelled' events should display in IOUDetails Modal #3028
Conversation
# Conflicts: # src/pages/iou/IOUTransactions.js
Hi @MonilBhavsar, I've noticed that sometimes duplicate transactions are displayed in the Details page. Once I've resolved this issue I'll remove the WIP marker. |
Alright @MonilBhavsar -- this is good to review on Monday. I fixed the duplicates issue. |
CC @iwiznia you may be interested in reviewing, but no pressure |
Ready for another review @MonilBhavsar @iwiznia:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Screenshots left to be added.
Added the screenshots |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
Details
As of the latest IOU Details issue, only IOU transaction events are displayed in the Modal, but not 'Settled' and 'Cancelled' events. This PR refactors the previous logic, enabling all reportComments of type IOU to be shown in the IOUDetails Modal.
This also fixes a bug caused by incorrect Number to String Javascript conversion.
Fixed Issues
Fixes https://github.com/Expensify/Expensify/issues/164703
Fixes https://github.com/Expensify/Expensify/issues/164704
Tests
NOTE: If you see the
This action was taken on old code and is not supported anymore
message, the IOU was created from out-of-date Auth code. Please ensure Auth is pulled and rebuilt, then create a new IOU to perform the test.5b. To find the
**reportID**
, click View Details and note the second route param5c. To locate the
**transactionID**
, open the Chrome dev tools Application/Local Storage tab, and search thereportID
from 5b. LocatereportIOUs_XYZ
where XYZ is the reportID.5d. Use one of the transactionID values in the API request in 5e
5e. Make the following API call, replacing authToken, email, accountId, reportID, & transactionID
5f. You should see a new 'Declined/Cancelled the $12.34 request' message, depending on which used rejected the transaction


6. Now, log out and log back in as userB
7. Once logged in, refresh the page to avoid this bug
8. Navigate to the chatReport with userA
QA Steps
Run above tests, but SKIP STEP 5!, as this is not yet possible.
Tested On
Screenshots
Web
Mobile Web
Desktop
iOS
Android