Page MenuHomePhabricator
Differential D10432

[lib/web/native] Convert getMentionTypeaheadUserSuggestions into a hook
ClosedPublic
Actions

Authored by rohan on Dec 21 2023, 1:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 1:19 PM
Unknown Object (File)
Tue, Dec 17, 9:06 AM
Unknown Object (File)
Mon, Dec 16, 1:37 AM
Unknown Object (File)
Fri, Dec 6, 6:09 PM
Unknown Object (File)
Nov 23 2024, 5:12 AM
Unknown Object (File)
Nov 14 2024, 4:15 PM
Unknown Object (File)
Nov 8 2024, 10:03 PM
Unknown Object (File)
Nov 8 2024, 7:18 PM
View All 60 Files
Subscribers
ashoat
tomek

Details

Reviewers
atul
ginsu
inka
Commits
rCOMM165d35ab1a3f: [lib/web/native] Convert getMentionTypeaheadUserSuggestions into a hook
Summary

This is some feedback provided in https://phab.comm.dev/D10387#inline-63506. This diff converts getMentionTypeaheadUserSuggestions into a hook that is consumed in both web and native.

Addressess ENG-6278

Depends on D10407

Test Plan

Made sure the @ mentioning for users and chats still looked fine, nothing seemed slow or out of place. Also opened up the Chrome console to make sure no React warnings were appearing.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rohan created this revision.Dec 21 2023, 1:37 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptDec 21 2023, 1:37 PM
rohan added inline comments.Dec 21 2023, 1:38 PM
lib/shared/mention-utils.js
137–140 ↗(On Diff #34928)

The conversion to a hook here is similar to the conversion I did for useMentionTypeaheadUserSuggestions above

Harbormaster completed remote builds in B25291: Diff 34928.Dec 21 2023, 1:55 PM
rohan requested review of this revision.Dec 21 2023, 1:55 PM
atul accepted this revision.Dec 21 2023, 4:26 PM
This revision is now accepted and ready to land.Dec 21 2023, 4:26 PM
Closed by commit rCOMM165d35ab1a3f: [lib/web/native] Convert getMentionTypeaheadUserSuggestions into a hook (authored by rohan). · Explain WhyDec 22 2023, 10:28 AM
This revision was automatically updated to reflect the committed changes.
rohan added a commit: rCOMM165d35ab1a3f: [lib/web/native] Convert getMentionTypeaheadUserSuggestions into a hook.

Revision Contents
Changeset List

PathSize
lib/
shared/
37 lines
native/
chat/
31 lines
web/
chat/
30 lines

Diff 35002

View Options

lib/shared/mention-utils.js

Loading...
View Options

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

Loading...
View Options

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

Loading...