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

chore(examples): update next example to Next 12 #3438

Merged
merged 6 commits into from
Apr 21, 2022

Conversation

Haroenv
Copy link
Contributor

@Haroenv Haroenv commented Apr 20, 2022

Summary

  • updated to latest next
  • removed css, jest, babel etc. configuration
  • convert from classes to hooks
  • change strategy of back button detection

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

@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 20, 2022

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:

Sandbox Source
react-instantsearch-app Configuration
hooks-example Configuration

Copy link
Member

@dhayab dhayab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's great! 👏

@netlify
Copy link

netlify bot commented Apr 20, 2022

Deploy Preview for react-instantsearch ready!

Name Link
🔨 Latest commit be11b39
🔍 Latest deploy log https://app.netlify.com/sites/react-instantsearch/deploys/625ffe693acc6b0008accb33
😎 Deploy Preview https://deploy-preview-3438--react-instantsearch.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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 });
Copy link
Member

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).

Copy link
Contributor Author

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

Copy link
Member

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.

@Haroenv Haroenv merged commit 391b9f4 into master Apr 21, 2022
@Haroenv Haroenv deleted the chore/update-next-example branch April 21, 2022 11:10
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.

3 participants