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

Fix documentation about hooks #30

Closed
cabol opened this issue Nov 8, 2018 · 0 comments
Closed

Fix documentation about hooks #30

cabol opened this issue Nov 8, 2018 · 0 comments

Comments

@cabol
Copy link
Owner

cabol commented Nov 8, 2018

Options :pre_hooks_strategy and :post_hooks_strategy were removed, now the strategy is passed along with the list of hooks, like so:

def pre_hooks do
  pre_hook =
    fn
      (result, {_, :get, _} = call) ->
        # do your stuff ...
      (result, _) ->
        result
     end

  {:async, [pre_hook]}
end

Fix the documentation in Nebulex.Hook module and also in the hooks.md guide – perhaps, check out other possible places.

cabol added a commit that referenced this issue Nov 12, 2018
@cabol cabol closed this as completed Nov 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant