ring-swagger

ring-swagger & compojure-api
2017-10-30T16:21:09.000598Z

@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).

2017-10-30T16:32:36.000418Z

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)

2017-10-30T16:36:07.000255Z

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

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

do you use Cursive?

2017-10-30T16:36:35.000093Z

Nevar 😉 :emacs:

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

haha. I'll test that when at laptop

2017-10-30T16:37:52.000371Z

Cool, thanks

2017-10-30T16:38:36.000559Z

Insanely good user support as usual 🙂

2017-10-30T16:39:59.000101Z

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

2017-10-30T16:43:24.000492Z

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