ring-swagger & compojure-api

@ikitommi is there a special reasoning why :- is required for e.g. query-params, but leads to errors if used for the body? It's pretty confusing that the handling is different from place to place

ikitommi 2017-10-30T16:30:25.000446Z

@mrchance just history. *-params use the plumbing/letk binding (good for asserting single params) while :body, :query and :headers use let binding (good for asserting the whole thing).


Ah, ok.. I thought it would be something along those lines. Not sure if the advantages outweigh the surprises, but probably hard to change now anyway 😉

ikitommi 2017-10-30T16:33:24.000638Z

I think it could be changed, but Colin would need to update the static analyzing part in Cursive too.

ikitommi 2017-10-30T16:34:12.000138Z

so, this would be better? :body [user :- User]

ikitommi 2017-10-30T16:35:22.000345Z

If that works oob with Cursive, it would be easy to add (just strip the :- in the middle position)


Hm, that's probably the best way then, just make it more tolerant

ikitommi 2017-10-30T16:36:20.000502Z

do you use Cursive?


Nevar 😉 :emacs:

ikitommi 2017-10-30T16:37:38.000111Z

haha. I'll test that when at laptop


Cool, thanks


Insanely good user support as usual 🙂


The colleague who had the problem asks if he can buy you a beer somewhere


I'd be interested too, you already helped a lot in the past @ikitommi