App: [HOLD for payment 2023-07-05] [$1000] Web - Chat - Mention list doesn't display when enter special key: '_-+=|\;`~ before:

@PauloGasparSv So will we add/accept these characters \ ’ ; `

Yes to: \ ; ` No to: '

So the list of accepted should be the following IMO: , / \ ? " { } [ ] ( ) & ^ % $ # < > ! * = ` ;

And let’s not add these characters there were discussed here at some point: _ + - @ ~ | '

@PauloGasparSv @abdulrahuman5196 You can see the current behavior on Slack and on Expensify for more information

I just checked on Slack if we enter some special key before @ or : and check whether the suggestion is displayed or not This is the result

Header	Slack	Expensify
_	yes	no
-	yes	no
+	yes (if after is 😃 and no (if after is @)	no
=	no	no
|	no	no
\	no	no
;	no	no
`	no	no
~	no	no
@	no	no

BZ checklist

The PR that introduced the bug has been identified. Link to the PR: The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

Not a regression. The characters in discussion was not previously added.

Determine if we should create a regression test for this bug. If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

No. This is a minor case and wouldn’t we beneficial to add a regression test.

@dukenv0307 In the proposed changes, Since this is a regex change, kindly mention how you would change the regex to make this working.

Add _-=|;`~ key to both SPECIAL_CHAR_OR_EMOJI and MENTION_REPLACER regex (don’t add + key because this key maybe exist in email)

This doesn’t provide complete information on how you would like to change the regex.

@PauloGasparSv I agree we should add a lower subset of character which don’t clash with existing experiences. Even if people want to add more we can do it in future if required. (Which i highly doubt will happen)