-
Notifications
You must be signed in to change notification settings - Fork 6
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
[vue3] beforeDestroy
has been renamed to beforeUnmount
#15
Comments
So do I just need to update this? Seems really strange that they would make a change breaking to vue2 when vue3 hooks make this irrelevant dont they? @stritti |
Yes but at least the naming is more clear now. The hook is not irrelevant. It is renamed: |
Yeah I'm gonna need advice on how to fix this (or a PR) because with my build pipeline I cant even build against vue 3 since the vue-template-compiler stopped being supported after vue 2.
And it doesnt work if I install Can I fix this issue just by having 2 methods on the mixin called beforeUnmount and beforeDestroy that do the exact same thing? Or does the current backwards compat try to call both of them? @stritti I would be willing to do that if so, but if not, the build pipeline is going to have to be redone, perhaps with parcel 2 beta |
The plugin is really useful. Do you have a plan to migrate to Vue3? |
@polinwei hey, thanks for the comment! Vue 3 requires you to rework how you think about libraries like mine. Good news though--there is what appears to be a great continuation to my library in vue 3! https://github.com/riderx/vue-timer-hook shoutouts to @riderx for doing what appears to be a great job on it |
Thank's @BrianRosamilia for the mention ! Your module was a inspiration to build something in vue 3 |
Using vue-crono together with vue3 there is a warning:
[Vue warn]: 'beforeDestroy' has been renamed to 'beforeUnmount'
The text was updated successfully, but these errors were encountered: