[HOLD for payment 2024-10-14] [$250] Search - Room mention in the message is not highlighted or linked when opened from Search

[IuliiaHerets]

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: 9.0.38-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers): applausetester+kh010901@applause.expensifail.com
Issue reported by: Applause Internal Team

Action Performed:

1. Go to staging.new.expensify.com
2. Go to a room.
3. Send a message containing #room mention.
4. Go to Search.
5. Go to Chat.
6. Click on the message sent in Step 3.

Expected Result:

The room mention in the message will be highlighted and linked when opened from Search.

Actual Result:

The room mention in the message is not highlighted or linked when opened from Search.

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

Bug6608683_1726739485892.bandicam_2024-09-19_17-47-50-037.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021838252249332966020
• Upwork Job ID: 1838252249332966020
• Last Price Increase: 2024-09-23
• Automatic offers:
• situchan | Reviewer | 104118455
• nkdengineer | Contributor | 104118456

Issue Owner

Current Issue Owner: @slafortune

[melvin-bot]

Triggered auto assignment to @slafortune (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

[IuliiaHerets]

We think that this bug might be related to #wave-control

[IuliiaHerets]

@slafortune FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

[nkdengineer]

Proposal

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

The room mention in the message is not highlighted or linked when opened from Search.

What is the root cause of that problem?

Currently, here the values of currentReportIDContext and currentReportID?.currentReportID are both -1 because we are not adding enough Provider and passing the correct reportID in the Search Page, which is why the value of isGroupPolicyReport is false, and we will not show the highlight for that message.

App/src/components/HTMLEngineProvider/HTMLRenderers/MentionReportRenderer/index.tsx

Line 63 in b0ce206

const currentReportIDValue = currentReportIDContext || currentReportID?.currentReportID;

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

We should add MentionReportContext.Provider for items and pass in the value mentionReportContextValue like we did here with

    const mentionReportContextValue = useMemo(() => ({currentReportID: report.reportID}), [report.reportID]);

1. We can add this provider to ChatListItem here to highlight text in list all chats

App/src/components/SelectionList/ChatListItem.tsx

Line 82 in b0ce206

<MultipleAvatars

2. Also add this provider to ReportActionItem here

App/src/pages/home/report/ReportActionItem.tsx

Line 905 in b0ce206

<PressableWithSecondaryInteraction

What alternative solutions did you explore? (Optional)

NA

result.mov

[melvin-bot]

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

[melvin-bot]

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