Page MenuHomePhabricator
Differential D6297

[1/n] Native Typeahead - Moving type decalration to lib
ClosedPublic
Actions

Authored by • przemek on Jan 18 2023, 8:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 10:11 AM
Unknown Object (File)
Tue, Oct 29, 5:28 AM
Unknown Object (File)
Tue, Oct 29, 5:28 AM
Unknown Object (File)
Tue, Oct 29, 5:28 AM
Unknown Object (File)
Tue, Oct 29, 5:27 AM
Unknown Object (File)
Tue, Oct 29, 5:23 AM
Unknown Object (File)
Fri, Oct 25, 5:46 PM
Unknown Object (File)
Sep 26 2024, 7:17 AM
View All 93 Files
Subscribers
ashoat
atul
ginsu
inka
kamil
michal
tomek

Details

Reviewers
inka
ginsu
kamil
michal
tomek
Commits
rCOMM9eb5b3df2c1f: [1/n] Native Typeahead - Moving type decalration to lib
Summary

We can use the same type both in web and native logic.
Thus moving it to shared typeahead utils.

Test Plan

Checked if web app still works

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• przemek created this revision.Jan 18 2023, 8:47 AM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptJan 18 2023, 8:47 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Jan 18 2023, 8:50 AM
Harbormaster failed remote builds in B15436: Diff 21045!
Harbormaster completed remote builds in B15436: Diff 21045.Jan 19 2023, 6:38 AM
• przemek requested review of this revision.Jan 19 2023, 6:38 AM
inka accepted this revision.Jan 20 2023, 3:54 AM
This revision is now accepted and ready to land.Jan 20 2023, 3:54 AM
• przemek added a child revision: D6298: [2/n] Native Typeahead - Adding utility file and native regex.Jan 24 2023, 1:07 PM
tomek accepted this revision.Jan 30 2023, 1:54 AM
ashoat updated this revision to Diff 22205.Feb 7 2023, 9:09 AM

Rebase

Harbormaster completed remote builds in B16234: Diff 22205.Feb 7 2023, 9:23 AM
Closed by commit rCOMM9eb5b3df2c1f: [1/n] Native Typeahead - Moving type decalration to lib (authored by • przemek, committed by ashoat). · Explain WhyFeb 7 2023, 2:37 PM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMM9eb5b3df2c1f: [1/n] Native Typeahead - Moving type decalration to lib.

Revision Contents
Changeset List

PathSize
lib/
shared/
5 lines
web/
chat/
6 lines
2 lines

Diff 22242

View Options

lib/shared/typeahead-utils.js

Loading...
View Options

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

Loading...
View Options

web/chat/typeahead-tooltip.react.js

Loading...