metabase / toucan

A classy high-level Clojure library for defining application models and retrieving them from a DB
Eclipse Public License 1.0
569 stars 49 forks source link

Should insert-many! call post-insert on inserted objects? #4

Open camsaul opened 7 years ago

camsaul commented 7 years ago

Seems like this makes sense since it calls pre-insert. People can always use simple-insert-many! instead to avoid this behavior

MageMasher commented 6 years ago

Hey @camsaul , is there any specific way I can help on this issue? I'm working on a new greenfield project and toucan is the silver bullet I've been looking for.

Do you want to discuss any of the tradeoffs?