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

Cancelling further hooks #236

Closed
eddyystop opened this issue Aug 1, 2017 · 2 comments
Closed

Cancelling further hooks #236

eddyystop opened this issue Aug 1, 2017 · 2 comments

Comments

@eddyystop
Copy link
Collaborator

Setting context.result prevents the DB call. It does not prevent the remaining before & after hooks from running.

Can we help to prevent hooks running?

You could set hook.cancelled = true and then wrap everything that should into a iff hook checking iff(hook => hook.cancelled, otherHooks)

@beeplin
Copy link
Contributor

beeplin commented Feb 17, 2018

#355

@eddyystop
Copy link
Collaborator Author

Consolidate in 355

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

No branches or pull requests

2 participants