ash-project / ash

A declarative, extensible framework for building Elixir applications.
https://www.ash-hq.org
MIT License
1.63k stars 220 forks source link

Documentation: filter checks #1519

Closed aidalgol closed 1 month ago

aidalgol commented 1 month ago

The example code for filter checks on the Policies page includes a comment that reads,

Filter checks don't have a context available to them"

But following the link right above that, Ash.Policy.FilterCheck, callback filter is documented as taking three arguments, actor, context, and options.

Also, is it correct for this example code snippet to be using expr in the body of filter, or is that only for inline filter-checks?

zachdaniel commented 1 month ago

Thanks for pointing this out :) It is correct for it to be using an expr in the body of the filter, but the other bit is wrong.