joy-framework / joy

A full stack web framework written in janet
https://joy.swlkr.com
MIT License
535 stars 30 forks source link

updated_at column is not updated on save #104

Open greenfork opened 1 year ago

greenfork commented 1 year ago

When creating a standard controller with joy create controller, the "patch" looks like this:

(defn accounts/patch [req]
  (when-let [account (account req)
             account (->> req account/body (merge account) db/save)]

    (if (saved? account)
      (redirect-to :accounts/index)
      (accounts/edit req (errors account)))))

It uses db/save from joy/db.janet which uses db/insert with ON CONFLICT UPDATE clause. But the column updated_at is only updated with db/update.

I personally don't think that the column updated_at even deserves to stay alive so it is not a problem for me, just reporting a thing that doesn't seem to work.