Page MenuHomePhabricator
Differential D8944

[web] Modify mention typeahead regex
ClosedPublic
Actions

Authored by • patryk on Aug 25 2023, 3:11 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 17, 7:06 PM
Unknown Object (File)
Fri, Jul 12, 11:53 AM
Unknown Object (File)
Fri, Jul 12, 11:53 AM
Unknown Object (File)
Fri, Jul 12, 11:53 AM
Unknown Object (File)
Fri, Jul 12, 11:53 AM
Unknown Object (File)
Fri, Jul 12, 10:27 AM
Unknown Object (File)
Fri, Jul 12, 6:47 AM
Unknown Object (File)
Tue, Jul 9, 4:25 PM
View All 42 Files
Subscribers
ashoat
tomek

Details

Reviewers
tomek
inka
rohan
Commits
rCOMM767028e40c32: [web] Modify mention typeahead regex
Summary

This diff modifies mention typeahead regex on web to support chat mentioning.

Depends on D8943.

Test Plan

Tested later in the stack.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• patryk created this revision.Aug 25 2023, 3:11 AM
• patryk held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 25 2023, 3:11 AM
• patryk edited the summary of this revision. (Show Details)Aug 25 2023, 3:16 AM
• patryk added reviewers: tomek, inka, rohan.
• patryk added a parent revision: D8943: [web] Refactor TypeaheadTooltip component.
• patryk added a child revision: D8945: [web] Show chats in mention typeahead tooltip.
• patryk edited the summary of this revision. (Show Details)Aug 25 2023, 3:23 AM
• patryk edited the test plan for this revision. (Show Details)
Harbormaster completed remote builds in B22080: Diff 30308.Aug 25 2023, 3:29 AM
• patryk published this revision for review.Aug 25 2023, 4:01 AM
• patryk added inline comments.
web/utils/typeahead-utils.js
22 ↗(On Diff #30308)

Potential drawbacks of this solution are described here.

rohan added a comment.Aug 25 2023, 6:19 AM

I think it'll be useful to add some unit tests for this regex, alongside the regex in D8834

inka added inline comments.Sep 6 2023, 1:45 AM
web/utils/typeahead-utils.js
22 ↗(On Diff #30308)

Is there any point is using an alternative here? Isn't oldValidUsernameRegexString a subset of validChatNameRegexString?

• patryk updated this revision to Diff 31255.Sep 19 2023, 3:48 AM

Remove oldValidUsernameRegexString.

Harbormaster completed remote builds in B22719: Diff 31255.Sep 19 2023, 4:05 AM
ashoat added a comment.Sep 19 2023, 7:27 AM
In D8944#263772, @rohan wrote:

I think it'll be useful to add some unit tests for this regex, alongside the regex in D8834

This comment doesn't appear to have been addressed

• patryk added a comment.Sep 22 2023, 7:29 AM

D9057

inka accepted this revision.Sep 22 2023, 7:32 AM
This revision is now accepted and ready to land.Sep 22 2023, 7:32 AM
Closed by commit rCOMM767028e40c32: [web] Modify mention typeahead regex (authored by • patryk). · Explain WhySep 28 2023, 5:43 AM
This revision was automatically updated to reflect the committed changes.
• patryk added a commit: rCOMM767028e40c32: [web] Modify mention typeahead regex.

Revision Contents
Changeset List

PathSize
web/
chat/
2 lines
utils/
4 lines

Diff 31512

View Options

web/chat/chat-input-bar.react.js

Loading...
View Options

web/utils/typeahead-utils.js

Loading...