-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update React Sample App according to new Headless State #54
Conversation
update StaticFilter to no longer use ID with headless public interface update function to transform static filters to displayableFilter to no longer account for ID Note: will update package path/version when new headless version is publish J=SLAP-1688 TEST=manual hook sample-app with local headless branch. Setup two static filters, and see that they work accordingly
…dless-react into dev/remove-id-in-filter
…dless-react into dev/remove-id-in-filter
…l address in another item
export function useAutocomplete(isVertical: boolean): [AutocompleteResult[], () => Promise<void>] { | ||
const answersActions = useAnswersActions(); | ||
const autocompleteNetworkIds = useRef({ latestRequest: 0, responseInState: 0 }); | ||
const [ autocompleteResults, setAutoCompleteResults ] = useState<AutocompleteResult[]>([]); |
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.
we have an item to be consistent about the capitzliation for AutoComplete vs Autocomplete but I think within this method it would be nice to be consistent as well
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.
oh yea, good catch. I made some changes to headless to have it lowercase, missed it here
Update React sample app and tests based on the updated headless state
SearchStatusState
andQueryState
SearchBar
to manage its own autocomplete results in component state. Added a useAutocomplete hookinputDropdown
in another itemAppliedFilter
andStaticFilter
to no longer use ID with headless public interfaceresetFacets
onhandleFilterOptionChange
J=SLAP-1693
TEST=manual
jest tests passed
smoke tested sample-app with new headless version