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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

onStateChange with outdated uiState #4421

Closed
eunjae-lee opened this issue May 28, 2020 · 0 comments 路 Fixed by #4555
Closed

onStateChange with outdated uiState #4421

eunjae-lee opened this issue May 28, 2020 · 0 comments 路 Fixed by #4555

Comments

@eunjae-lee
Copy link
Contributor

馃悰 Bug description

onStateChange doesn't receive the latest uiState when it's refined from connectConfigure.

馃攳 Bug reproduction

Steps to reproduce the behavior:

  1. Go to https://codesandbox.io/s/friendly-wozniak-48i5g?file=/src/app.js
  2. Click on "click me" button
  3. See the console log
  4. Check the configure key has empty object as a value. (It's supposed to be configure: { hitsPerPage: 4 }

Live reproduction:

https://codesandbox.io/s/friendly-wozniak-48i5g?file=/src/app.js

馃挱 Expected behavior

It should deliver the latest uiState.

Environment

  • OS: macOS 10.15.3
  • Browser: Firefox 76.0.1 (64-bit)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(configure): pass the latest state to onStateChange algolia/instantsearch
1 participant
@eunjae-lee