-
Notifications
You must be signed in to change notification settings - Fork 9
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
Separate recent searches in verticals #242
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
alextaing
commented
Jul 19, 2022
Are there any suggestions on writing automated tests for this item? Is it something that is best done with unit tests, or is its scope a little too wide? |
oshi97
reviewed
Jul 19, 2022
oshi97
reviewed
Jul 19, 2022
Current coveralls error is due to 504 gateway error, will try to rerun coveralls in an hour |
oshi97
reviewed
Jul 19, 2022
oshi97
approved these changes
Jul 19, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯 💯 💯
Merged
ElemelonWind
added a commit
that referenced
this pull request
Jul 20, 2022
## Version 0.2.0-beta.243 `@yext/answers-react-components` is now `@yext/search-ui-react`! ### Changes - Exported a string representing the content path to files in component lib that uses tailwind classnames. (#193) - Trimmed/edited various CSS classes interfaces. (#189, #190, #191, #194, #196, #197, #198, #200, #202) - Made hovering over `FilterSearch` autocomplete suggestions trigger a highlight on the selected option and fixed its width/padding. (#203, #205) - Removed `allowPagination` prop from `VerticalResults` (#217, #219) - Updated `StandardFacets` to optionally display the number of results associated with each facet option. (#226, #248) - Added a 'clear filter' feature for 'clear min and max' button in `RangeInput`. (#227) - Added a range to the `ResultCount` that describes the range of results currently displayed by `Pagination`. (#228) - Replaced `onSubmit` with `dropdownItemProps` in `RenderEntityPreviews` interface. (#225, #238) - Separated recent searches in verticals to keep their recent search histories separate. (#242) - Removed links from `AlternativeVerticals` and deleted `UniversalLink` interface. (#229) - `@yext/answers-headless-react` has been renamed to `@yext/search-headless-react`, and has been replaced by such as a peer dependency. (#243) ### Bugfixes - When `label` is undefined, `FilterSearch` does not display default 'filter' label (#201) - Fixed `DropdownMenu` tabbing and selecting behavior (#206, #207) - Fixed a bug where clicking the `ApplyFiltersButton` did not reset the pagination offset. (#213) - Fixed linebreak bug in `resultsCount` (#218) - Fixed a bug where range input static filters were removed when selecting/de-selecting facet options with different `fieldId`s. (#224) - Fixed `LocationBias` mobile styling (#221) - Fixed `ThumbsFeedback` issues of not having a default width and height, as well as rotation issue with the thumbs-down button. (#233) - Fixed `inputPrefix` collision issue and tooltip location in `RangeInput`. (#232, #234) - Fixed a bug where facets were not being reset when applying a filter from `FilterSearch` with `searchOnSelect=true`. (#236)
yen-tt
pushed a commit
that referenced
this pull request
Sep 28, 2022
This PR allows recent searches to show in verticals and keeps their recent search histories separate. J=SLAP-2241 TEST=manual, auto The test site was used to check that recent search histories in different verticals (and in universal) were all kept separate. Jest tests were written for the `useRecentSearches` hook, testing that recent searches appear, that they stay on their own verticals, and that they can be cleared using the returned `clearRecentSearches` function.
yen-tt
pushed a commit
that referenced
this pull request
Sep 28, 2022
## Version 0.2.0-beta.243 `@yext/answers-react-components` is now `@yext/search-ui-react`! ### Changes - Exported a string representing the content path to files in component lib that uses tailwind classnames. (#193) - Trimmed/edited various CSS classes interfaces. (#189, #190, #191, #194, #196, #197, #198, #200, #202) - Made hovering over `FilterSearch` autocomplete suggestions trigger a highlight on the selected option and fixed its width/padding. (#203, #205) - Removed `allowPagination` prop from `VerticalResults` (#217, #219) - Updated `StandardFacets` to optionally display the number of results associated with each facet option. (#226, #248) - Added a 'clear filter' feature for 'clear min and max' button in `RangeInput`. (#227) - Added a range to the `ResultCount` that describes the range of results currently displayed by `Pagination`. (#228) - Replaced `onSubmit` with `dropdownItemProps` in `RenderEntityPreviews` interface. (#225, #238) - Separated recent searches in verticals to keep their recent search histories separate. (#242) - Removed links from `AlternativeVerticals` and deleted `UniversalLink` interface. (#229) - `@yext/answers-headless-react` has been renamed to `@yext/search-headless-react`, and has been replaced by such as a peer dependency. (#243) ### Bugfixes - When `label` is undefined, `FilterSearch` does not display default 'filter' label (#201) - Fixed `DropdownMenu` tabbing and selecting behavior (#206, #207) - Fixed a bug where clicking the `ApplyFiltersButton` did not reset the pagination offset. (#213) - Fixed linebreak bug in `resultsCount` (#218) - Fixed a bug where range input static filters were removed when selecting/de-selecting facet options with different `fieldId`s. (#224) - Fixed `LocationBias` mobile styling (#221) - Fixed `ThumbsFeedback` issues of not having a default width and height, as well as rotation issue with the thumbs-down button. (#233) - Fixed `inputPrefix` collision issue and tooltip location in `RangeInput`. (#232, #234) - Fixed a bug where facets were not being reset when applying a filter from `FilterSearch` with `searchOnSelect=true`. (#236)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR allows recent searches to show in verticals and keeps their recent search histories separate.
J=SLAP-2241
TEST=manual, auto
The test site was used to check that recent search histories in different verticals (and in universal) were all kept separate. Jest tests were written for the
useRecentSearches
hook, testing that recent searches appear, that they stay on their own verticals, and that they can be cleared using the returnedclearRecentSearches
function.