Skip to content
This repository has been archived by the owner on Dec 30, 2022. It is now read-only.

fix(ssr): allow getWidgetState to be empty for createURL #805

Merged
merged 2 commits into from
Jun 18, 2020

Conversation

Haroenv
Copy link
Contributor

@Haroenv Haroenv commented Jun 18, 2020

The check was on the widget itself, instead of on the current iteration.

See this for original code which didn't have the issue:

https://github.com/algolia/instantsearch.js/blob/ada37ae054275d46cc2edaee0081562b6c35cbc8/src/widgets/index/index.ts#L101-L115

@Haroenv Haroenv requested review from yannickcr and eunjae-lee June 18, 2020 11:37
@Haroenv Haroenv marked this pull request as ready for review June 18, 2020 11:37
@Haroenv Haroenv merged commit 0528b55 into master Jun 18, 2020
@Haroenv Haroenv deleted the fix/ssr-create-url branch June 18, 2020 12:07
Haroenv added a commit to algolia/instantsearch that referenced this pull request Dec 28, 2022
…-instantsearch#805)

* fix(ssr): allow getWidgetState to be empty for createURL

This must have been caused by a wrong copy-paste

* test(ssr): add case for createURL
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants