[HOLD for payment 2024-03-07] [LOW] [P2P] [$500] IOU - Delay in updating green dot and total amount when the net amounts cancel each other out for 2 users

[lanitochka17]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Version Number: 1.4.19-1
Reproducible in staging?: Y
Reproducible in production?: Y
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
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

Action Performed:

Precondition: User A has requested money from User B

1. Log in as User B
2. Go to 1:1 DM with User A > Click IOU preview
3. Request the same amount from User A to make the net owe become 0
4. Note the the green dot for the main chat in LHN
5. Return to main chat
6. Return to the IOU report
7. Delete the requested amount created in Step 3

Expected Result:

In Step 3, once the net owe becomes 0, the green dot for the main chat in LHN will disappear
In Step 7, the new amount should update instantly

Actual Result:

In Step 3, once the net owe becomes 0, the green dot for the main chat in LHN does not disappear, unless the main chat is visited
In Step 7, the total amount remains at 0, unless the report is revisited

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6328606_1703873451494.20231229_081427.1.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~017d07091eedfc5a43
• Upwork Job ID: 1740800068445937664
• Last Price Increase: 2023-12-29
• Automatic offers:
• shubham1206agra | Reviewer | 28079575
• tienifr | Contributor | 28079576

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~017d07091eedfc5a43

[melvin-bot]

Triggered auto assignment to @jliexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[melvin-bot]

Bug0 Triage Checklist (Main S/O)

• This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
• This bug is not a duplicate report (check E/App issues and #expensify-bugs)
  • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
• This bug is reproducible using the reproduction steps in the OP. S/O
  • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
  • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
• This issue is filled out as thoroughly and clearly as possible
  • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
• I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @shubham1206agra (External)

[abzokhattab]

Proposal

Please re-state the problem that we are trying to solve in this issue.

green dot should be removed when the net amount is 0 in iou

What is the root cause of that problem?

inside the requiresAttentionFromCurrentUser we define if the current report should show the green dot or not and currently we dont consider the case where the net is zero

App/src/libs/ReportUtils.ts

Line 1596 in 48953d0

function requiresAttentionFromCurrentUser(optionOrReport: OnyxEntry<Report> | OptionData, parentReportAction: EmptyObject | OnyxEntry<ReportAction> = {}) {

What changes do you think we should make in order to solve the problem?

we need to add the following condition:

    if ( optionOrReport?.iouReportAmount === 0) {
        return false;
    }

also, we might add a condition that hasOutstandingChildRequest is true to make sure that it has child iou requests but I believe this is redundant

for the delay in the message in point 7, this is a backend pusher issue

[paultsimura]

Proposal

Please re-state the problem that we are trying to solve in this issue.

The green dot of the parent chat report doesn't update immediately.

What is the root cause of that problem?

This is happening because we do not update the hasOutstandingChildRequest of the parent 1:1 report when requesting the money.

As a result, the chat report's green dot is displayed because of this check:

App/src/libs/ReportUtils.ts

Lines 1614 to 1617 in a74cc7a

	// Has a child report that is awaiting action (e.g. approve, pay, add bank account) from current user
	if (optionOrReport.hasOutstandingChildRequest) {
	return true;
	}

What changes do you think we should make in order to solve the problem?

In IOU.buildOnyxDataForMoneyRequest, we need to optimistically set the chat report's hasOutstandingChildRequest to false if the IOU Report's total will become 0 after the request is added:

hasOutstandingChildRequest: iouReport.total !== 0,

App/src/libs/actions/IOU.js

Lines 307 to 319 in 433bc47

	const optimisticData = [
	{
	// Use SET for new reports because it doesn't exist yet, is faster and we need the data to be available when we navigate to the chat page
	onyxMethod: isNewChatReport ? Onyx.METHOD.SET : Onyx.METHOD.MERGE,
	key: `${ONYXKEYS.COLLECTION.REPORT}${chatReport.reportID}`,
	value: {
	...chatReport,
	lastReadTime: DateUtils.getDBTime(),
	lastMessageTranslationKey: '',
	iouReportID: iouReport.reportID,
	...(isNewChatReport ? {pendingFields: {createChat: CONST.RED_BRICK_ROAD_PENDING_ACTION.ADD}} : {}),
	},
	},

We need to do the same on the money request removal:

        ...(shouldDeleteIOUReport
            ? [
                  {
                      onyxMethod: Onyx.METHOD.MERGE,
                      key: `${ONYXKEYS.COLLECTION.REPORT}${chatReport.reportID}`,
                      value: {
                          hasOutstandingChildRequest: false,
                          iouReportID: null,
                          lastMessageText: ReportActionsUtils.getLastVisibleMessage(iouReport.chatReportID, {[reportPreviewAction.reportActionID]: null}).lastMessageText,
                          lastVisibleActionCreated: lodashGet(ReportActionsUtils.getLastVisibleAction(iouReport.chatReportID, {[reportPreviewAction.reportActionID]: null}), 'created'),
                      },
                  },
              ]
            : [
                {
                    onyxMethod: Onyx.METHOD.MERGE,
                    key: `${ONYXKEYS.COLLECTION.REPORT}${chatReport.reportID}`,
                    value: {
                        hasOutstandingChildRequest: updatedIOUReport.total !== 0,
                    },
                },
            ]),

App/src/libs/actions/IOU.js

Lines 2422 to 2435 in 433bc47

	...(shouldDeleteIOUReport
	? [
	{
	onyxMethod: Onyx.METHOD.MERGE,
	key: `${ONYXKEYS.COLLECTION.REPORT}${chatReport.reportID}`,
	value: {
	hasOutstandingChildRequest: false,
	iouReportID: null,
	lastMessageText: ReportActionsUtils.getLastVisibleMessage(iouReport.chatReportID, {[reportPreviewAction.reportActionID]: null}).lastMessageText,
	lastVisibleActionCreated: lodashGet(ReportActionsUtils.getLastVisibleAction(iouReport.chatReportID, {[reportPreviewAction.reportActionID]: null}), 'created'),
	},
	},
	]
	: []),

What alternative solutions did you explore? (Optional)

[tienifr]

Proposal

Please re-state the problem that we are trying to solve in this issue.

1. In Step 3, once the net owe becomes 0, the green dot for the main chat in LHN does not disappear, unless the main chat is visited
2. In Step 7, the total amount remains at 0, unless the report is revisited

What is the root cause of that problem?

1. We're not optimistically setting hasOutstandingChildRequest to the correct value after user B creates a money request that offsets the initial request from user A
2. We have a regression from this TS migration, as we can see, in here, we're checking if iouReportUpdate.total is truthy before making the calculation. This is wrong because the total could be 0, we should still calculate in that case. This is probably because when migrating to TS, we have to make sure the total is not undefined to avoid type issue, so if (iouReportUpdate.total) was added. This leads to the amount will not optimistically change when we delete the money request.

What changes do you think we should make in order to solve the problem?

1. We should set hasOutstandingChildRequest in optimistic data of the chat report here

hasOutstandingChildRequest: iouReport.managerID === userAccountID && iouReport.total !== 0,

We have to compare managerID === userAccountID as well because hasOutstandingChildRequest happens only if the current user owes the money, if the user is owed money, it should not be true and GBR should not show, even though in this case total is still more than 0. Can see more explanation here

We should do the same when deleting money request as well here (checking both iouReport.managerID and iouReport.total)

2. Instead of checking if iouReportUpdate.total is truthy here, we should check that iouReportUpdate.total is a number.

if (typeof iouReportUpdate.total === 'number') {

(or we can check that iouReportUpdate.total is specifically not null/undefined)

What alternative solutions did you explore? (Optional)

NA