[4.5/n] Native Typeahead - Refactoring to the common utility function
ClosedPublic
Actions

Authored by • przemek on Jan 27 2023, 2:24 PM.

Details

Reviewers

inka
ginsu
kamil
michal
tomek

Commits

rCOMM9a5ad243ee32: [4.5/n] Native Typeahead - Refactoring to the common utility function

Summary

Refactoring some web code into lib function that will be used in native side as well.

Test Plan

Checked if app builds.
Checked if web typeahead still works corretly

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• przemek created this revision.Jan 27 2023, 2:24 PM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptJan 27 2023, 2:24 PM

• przemek added a parent revision: D6300: [4/n] Native Typeahead - Add user search and thread members to props.Jan 27 2023, 2:37 PM

• przemek added a child revision: D6367: [5/n] Native Typeahead - Add selection state to chat input bar.

Harbormaster completed remote builds in B15721: Diff 21442.Jan 27 2023, 2:38 PM

• przemek requested review of this revision.Jan 27 2023, 2:38 PM

tomek accepted this revision.Jan 30 2023, 10:51 AM

This revision is now accepted and ready to land.Jan 30 2023, 10:51 AM

Rebase

Harbormaster completed remote builds in B16239: Diff 22210.Feb 7 2023, 5:36 PM

Closed by commit rCOMM9a5ad243ee32: [4.5/n] Native Typeahead - Refactoring to the common utility function. · Explain WhyFeb 7 2023, 10:37 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM9a5ad243ee32: [4.5/n] Native Typeahead - Refactoring to the common utility function.

Revision Contents
Changeset List

Path

Size

lib/

shared/

typeahead-utils.js

25 lines

web/

utils/

typeahead-utils.js

24 lines

Diff 22210

View Options

lib/shared/typeahead-utils.js