Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add keyword search #1129

Merged
merged 6 commits into from
Jul 18, 2019
Merged

Add keyword search #1129

merged 6 commits into from
Jul 18, 2019

Conversation

OtterleyW
Copy link
Contributor

@OtterleyW OtterleyW commented Jul 5, 2019

Add keyword search components to SearchFilters.js and SearchFiltersMobile.js. On mobile live edit is in use so we need to use debounce in FilterForm.js so that the query is made after the user has stopped typing.

On desktop:
keyword-search-desktop

On mobile:
keyword-search-mobile

@OtterleyW OtterleyW force-pushed the keyword-search-filter-desktop branch 3 times, most recently from 31e31c9 to e71a108 Compare July 9, 2019 10:06
@OtterleyW OtterleyW changed the title WiP: use keyword search Add keyword search Jul 12, 2019
@Gnito Gnito force-pushed the keyword-search-filter-desktop branch 2 times, most recently from ca501e8 to 5ab7610 Compare July 16, 2019 18:54
@Gnito Gnito temporarily deployed to sharetribe-starter-app July 16, 2019 18:58 Inactive
@Gnito Gnito temporarily deployed to sharetribe-starter-app July 17, 2019 12:41 Inactive
@Gnito Gnito force-pushed the keyword-search-filter-desktop branch from aedfea5 to b572284 Compare July 17, 2019 15:47
@Gnito Gnito force-pushed the keyword-search-filter-desktop branch 2 times, most recently from 8c7ae31 to 500155c Compare July 18, 2019 08:58
@Gnito Gnito temporarily deployed to sharetribe-starter-app July 18, 2019 12:15 Inactive
@Gnito Gnito force-pushed the keyword-search-filter-desktop branch from 500155c to 8cf99fd Compare July 18, 2019 14:38
@Gnito Gnito merged commit bd76364 into master Jul 18, 2019
@Gnito Gnito deleted the keyword-search-filter-desktop branch July 18, 2019 14:50
@Gnito Gnito mentioned this pull request Jul 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants