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.

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

Consider removing setImmediate call #37

Open
ivankoleda opened this issue Nov 8, 2020 · 0 comments
Open

Consider removing setImmediate call #37

ivankoleda opened this issue Nov 8, 2020 · 0 comments

Comments

@ivankoleda
Copy link

setImmediate is deprecated and it may work fine with some tools like CRA, but it does not work out of the box with nextjs. So anyone who is using this library with build tools have to polyfill it. Seem like this call may not be wrapped in setImmediate.

     setImmediate(wizard.next)

https://github.com/zaguiini/formik-wizard/blob/master/src/index.tsx#L82
Please, correct me if I'm wrong and there is a reason for using setImmediate. Otherwise it may be removed and this will help people who are not able to use this library out of the box.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant