From a92d296d58849adc18c33fb78393187c15c387e9 Mon Sep 17 00:00:00 2001 From: laviro <1ronlavi@gmail.com> Date: Thu, 12 Apr 2018 13:20:13 +0300 Subject: [PATCH] Add 'query' param, which is the text string entered by the user, to the onSearch function. Might be usefull with filtering on heavy requests. --- src/components/AutoComplete/AsyncAutoComplete.js | 5 +++-- src/components/AutoComplete/AutoComplete.stories.js | 3 ++- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/components/AutoComplete/AsyncAutoComplete.js b/src/components/AutoComplete/AsyncAutoComplete.js index cee11eebe73..7568c48f269 100644 --- a/src/components/AutoComplete/AsyncAutoComplete.js +++ b/src/components/AutoComplete/AsyncAutoComplete.js @@ -12,9 +12,10 @@ class AsyncAutoComplete extends React.Component { onSearchEnd = options => this.setState({ options, isLoading: false }); - handleSearch = () => { + // query is the text string entered by the user. + handleSearch = query => { this.onSearchStart(); - Promise.resolve(this.props.onSearch()).then(options => + Promise.resolve(this.props.onSearch(query)).then(options => this.onSearchEnd(options) ); }; diff --git a/src/components/AutoComplete/AutoComplete.stories.js b/src/components/AutoComplete/AutoComplete.stories.js index ede589da1ed..289a9207333 100644 --- a/src/components/AutoComplete/AutoComplete.stories.js +++ b/src/components/AutoComplete/AutoComplete.stories.js @@ -177,7 +177,8 @@ AutoCompleteStories.addWithInfo('Use With Forms', () => { }); AutoCompleteStories.addWithInfo('Using Async Calls', () => { - const handleSearch = () => + // query is the text string entered by the user. + const handleSearch = query => fetch('https://api.github.com/repos/patternfly/patternfly-react/forks') .then(response => response.json()) .then(forks => forks.map(fork => fork.owner));