-
Notifications
You must be signed in to change notification settings - Fork 383
chore(examples): update next example to Next 12 #3438
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit be11b39:
|
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.
That's great! 👏
Co-authored-by: François Chalifour <[email protected]>
Co-authored-by: François Chalifour <[email protected]>
✅ Deploy Preview for react-instantsearch ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
debouncedSetState.current = setTimeout(() => { | ||
const href = searchStateToURL(nextSearchState); | ||
|
||
router.push(href, href, { shallow: true }); |
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.
This seems to overwrite the router history, not to append (see preview).
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.
can you screen record?
Untitled.mov
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.
It seems to happen when you open the CodeSandbox app in a tab. It might be another CodeSandbox bug.
Summary
Result
Example is updated
see https://github.com/algolia/react-instantsearch/discussions/3434
Before this PR gets merged, the documentation should be updated, as it references this sandbox I think