om

Please ask the channel first, not @dnolen directly!
levitanong 2017-03-13T00:50:51.058040Z

@sova Unless there is some special requirement I would use margins or padding.

2017-03-13T03:27:15.897330Z

any suggestions on composing a sub-query that has params?

2017-03-13T03:28:39.904819Z

i'm currently just using the top level key of the sub component in the parent without the query to pass to the reconciler

2017-03-13T03:29:54.911307Z

(defui AutoCompleter
  static om/IQueryParams
  (params [_] {:query ""})
  static om/IQuery
  (query [_]
    '[(:search/results {:query ?query})])
is the child

2017-03-13T03:30:26.914523Z

and the parent is currently:

(defui Hello
  static om/IQuery
  (query [this]
    `[:wp/query-text {:wp/preview-list ~(om/get-query Preview)}
     :search/results])

2017-03-13T03:30:44.916178Z

then i just handle the two cases differently in the parser

2017-03-13T03:31:22.919481Z

when the query is present I pass it on to the remote to make a request, when it's not I just do a read and pass :search/results back down to AutoCompleter

2017-03-13T15:20:44.899153Z

I added children to the docs: https://github.com/omcljs/om/wiki/Documentation-(om.next)#children I didn't know about this, I stumbled across it reading the untangled source.

đź‘Ť 4
ag 2017-03-13T17:55:37.516860Z

@danielstockton the doc explains “get” part, but doesn’t actually say anything how to “pass” children in

2017-03-13T17:58:56.593622Z

Well, it says trailing args. There are lots of functions you could say that for I think.

ag 2017-03-13T18:01:51.663705Z

“trailing args” might be a bit confusing for starters

2017-03-13T18:05:43.751474Z

I agree, it's not the only thing like that in the docs though. For me it would have been enough to know it exists. They're editable by anyone so feel free to add more detail.

ag 2017-03-13T18:59:50.913310Z

Would Om.Next support Fiber out of the box?

anmonteiro 2017-03-13T19:01:00.939396Z

yes

1
🦜 1
anmonteiro 2017-03-13T19:01:32.951114Z

@ag just need to upgrade the react dep and this minimal patch https://github.com/anmonteiro/om/commit/686921531c7bcbbd75804e6b9240b12d92eaaa5f#diff-ee424e6559581b5b893371373a777e5eR188

anmonteiro 2017-03-13T19:01:49.956809Z

just the line highlighted there ^