[HOLD for payment 2024-05-09] Stop sending <mention-report> when not in a policy room

[rlinoz]

When we add a comment to a report we parse it before sending to the backend with ExpensiMark here.

We added the logic to parse #room-name to <mention-report>#room-name</mention-report> here, but since we only allow report mentions to happen inside policy rooms, we should prevent the parsing from happening for messages sent outside of a policy room.

Issue Owner

Current Issue Owner: @greg-schroeder

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~013f795318d2531edc
• Upwork Job ID: 1787964711355367424
• Last Price Increase: 2024-05-07

[robertKozik]

Hi, can you assign me to this issue?

[robertKozik]

PR for parser is ready

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.69-2 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Stop sending <mention-report> when not in a policy room #40685

If no regressions arise, payment will be issued on 2024-05-09. 🎊

For reference, here are some details about the assignees on this issue:

• @robertKozik does not require payment (Contractor)

[melvin-bot]

Triggered auto assignment to @greg-schroeder (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.

[rlinoz]

@greg-schroeder, @ahmedGaber93 was the PR reviewer.

[greg-schroeder]

Okay got it - set for payment on 5/9

[greg-schroeder]

Going to add Internal label just to create an upwork job

[melvin-bot]

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

[melvin-bot]

Current assignee @ahmedGaber93 is eligible for the Internal assigner, not assigning anyone new.

[greg-schroeder]

Offer sent to @ahmedGaber93 ahead of 5/9 payment date

[ahmedGaber93]

@greg-schroeder offer is accepted, Thanks!

[greg-schroeder]

paid