Clojurians
#untangled
<
2016-03-08
>

This page is not created by, affiliated with, or supported by Slack Technologies, Inc.

tony.kay18:03:00

@ethangracer: Did you change anything with respect to query parameter parsing in UC?

tony.kay18:03:13

I’ve got a regression where parameters are not making it to the server

tony.kay18:03:30

nvm…I figured it out

tony.kay18:03:55

So, I think it turns out that params substitution via load on the query is a bad idea (tm)

tony.kay18:03:53

or at least as conceived isn’t well formed

tony.kay18:03:57

@ethangracer: Are you guys using params option of load-collection/field on survey?

ethangracer18:03:12

we are currently, yes, but only in one place. wouldn’t be a big deal to change it

ethangracer18:03:56

I agree it’s definitely not well formed, we can really only add parameters to the first child of a query

ethangracer18:03:12

it’s actually only useful for load-field, in that case

tony.kay18:03:34

yeah, I think the Om query parameter fixes that A.M. did recently will be more useful

tony.kay18:03:02

doing a bit of research/testing…but I don’t think that code is in alpha31