-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add delay to nap present()s #4
Comments
But isn't it possible for the resultant action of nap-a to change which nap runs next ? |
Great idea! I just created #8 for that. Let's leave this one for discussion on the delay (should it be implemented anyway). Here, I've pictured the DevTools executing a sequence of naps with 500ms intervals. This seems important for chained/consecutive naps. Question being: How important/who often do we have these? |
Difficult to know without a real world example ?
|
Precisely. Let's come back to this in the future. |
I believe a small delay for nap
present()
would be great for the developer to understand what's going on.How important this is? I'm not sure.
Simply using
setTimeout
has shown to produce a lot of problems with nap. I believe we should buffer allpresent()
and process sequentially. If we do this, we could even stop in time right before nap'spresent()
. Imagine how cool it would be to be able to change the code and see what nap will do, before actually updating the store.The text was updated successfully, but these errors were encountered: