-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Blueprints action destroy is not called with meta #5630
Comments
@Alessy Thanks for posting! We'll take a look as soon as possible. In the mean time, there are a few ways you can help speed things along:
Please remember: never post in a public forum if you believe you've found a genuine security vulnerability. Instead, disclose it responsibly. For help with questions about Sails, click here. |
@Alessy - Thanks for spotting that, I'll bring it up to the team. If you're reaching a point were Blueprints is no longer working out of the box, it's ok to go ahead and write a backend action instead of using blueprint actions as a workaround 👍 |
Same issue here, it was called before. |
@dukewan As a workaround, have you tried using a backend action instead of a blueprint action for your project? |
@johnabrams7 I didn’t, I used the beforeDestroy hook instead which is not ideal but works for me. |
[fixes #5630] Update destroy blueprint action so that `afterDestroy` lifecycle callbacks will still run by default
@Alessy @dukewan @johnabrams7 just fixed this in #6847 |
Node version: 8.16.0
Sails version (sails): 1.2.2
ORM hook version (sails-hook-orm): 2.1.1
DB adapter & version (e.g. sails-mysql): 1.0.1
From documentation:
Blueprints action destroy don't include meta object so afterDestroy lifecycle callback don't run.
The text was updated successfully, but these errors were encountered: