-
-
Notifications
You must be signed in to change notification settings - Fork 78
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
Lifecycle methods #31
Conversation
Like this, but what is impact on perf? Calling a lot of noop? Also we need docs for this stuff. |
For the noop, the jit normally see that the function call does nothing and remove it. It is actually faster than |
I just found out that this build sometime throw the |
Okay. |
Oh nevermind about the error, it was because of something else in my app, not because of that patch. |
@Madumo sorry for delay) Now I'm working on v5 with full lifecycle events (see next branch) |
Like we talked on issue #27, there is no real lifecycle hooks right now. I just added some:
beforeUpdate and afterUpdate are both called with the update data.