ZennerIoT/ex_audit

Callback Repo.reload missing

h4cc opened this issue · 2 comments

h4cc commented

The Repo.reload and Repo.reload! callbacks seem to miss:

https://hexdocs.pm/ecto/Ecto.Repo.html#c:reload/2

Not sure why these should need to be implemented, ex_audit's repo use macro only overrides a handful of functions from the original ecto repo, so all newly added repo functions should be available.

Since we do this when we build a repo:

defmodule MyApp.Repo

  use Ecto.Repo, otp_app: :my_app
  use ExAudit.Repo, ...
end

Functions not overridden by use ExAudit.Repo "fall through" into the final MyApp.Repo module.

Actually, ex_audit needs to support ecto 3.5 to be able to use Repo.reload/2 - #61