[HOLD for payment 2023-08-10] [$2000] 'Date' section is not grayed out and the cursor does not change on the IOU report

[kbecciv]

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:

1. Request Money from any user
2. Go to the IOU report
   Notice that the Amount and Description are grayed out But not the 'Date' section. And the cursor does not change.

Expected Result:

The 'Date' section is grayed out and the cursor changes

Actual Result:

The 'Date' section is not grayed out and the cursor does not change on the IOU report

Workaround:

Unknown

Platforms:

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

• Android / native
• Android / Chrome
• iOS / native
• iOS / Safari
• MacOS / Chrome / Safari
• MacOS / Desktop

Version Number: 1.3.39.5
Reproducible in staging?: y
Reproducible in production?: n
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

Test71_Notgrayedout-1.mp4

Recording.3567.mp4

Expensify/Expensify Issue URL:
Issue reported by: @daveSeife
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689115720723609

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0184afd377a2246d04
• Upwork Job ID: 1684480022772416512
2023-07-31
• Automatic offers:
• aimane-chnaif | Reviewer | 25857860
• c3024 | Contributor | 25857863
• daveSeife | Reporter | 25857866

[OSBotify]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[melvin-bot]

Triggered auto assignment to @Julesssss (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[c3024]

Proposal

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

Date in money request report is not greyed out

What is the root cause of that problem?

In amount and description we have disabled={isSettled} but it is missing in the date in MoneyRequestView page
Date field does not have disabled = {isSettled}

<MenuItemWithTopDescription
                description={translate('common.date')}
                title={formattedTransactionDate}
                // shouldShowRightIcon={!isSettled}
                // onPress={() => Navigation.navigate(ROUTES.getEditRequestRoute(props.report.reportID, CONST.EDIT_REQUEST_FIELD.DATE))}
            />

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

We should add disabled = {isSettled} for that.

<MenuItemWithTopDescription
                description={translate('common.date')}
                title={formattedTransactionDate}
                disabled={isSettled}
                // shouldShowRightIcon={!isSettled}
                // onPress={() => Navigation.navigate(ROUTES.getEditRequestRoute(props.report.reportID, CONST.EDIT_REQUEST_FIELD.DATE))}
            />

We can do the same in MoneyRequestDetails as well just in case.

What alternative solutions did you explore? (Optional)

[Julesssss]

Definitely not a blocker. Checking it off the list.

[Julesssss]

Odd, I'm not able to reproduce on production (web). I can see here that @kbecciv was able to though 😕