dybr

Bill wrote a bunch of good notes on my PR, I fixed it all. he approved.

But then I continued improving the code (removing calls like `something.somefun(.....,None, None, None, None)`) etc.

It turned out there's about 6 files updated. So I asked Bill if he does not mind re-reviewing, but later figured I'll send it in another PR. And asked Esteban to add his review.

We were supposed to have a sprint postmortem, but there were just three of us, so we dropped it.

I found pretty weird sql injection in a bunch of code, but it turned out (with thanks to Brad and Mike and Bill) that Doobie processes all these things properly, and creates a correct sql statement, so everything's fine.

By the end of the day there was.a little bit of a panic, where one of our services started rebooting. Turned out it was because it was returning 500 all the time. And why was it? Because the service we call decided not to accept userId. It's a known feature of that team. They deployed it today, in the afternoon. Good timing.