om

Please ask the channel first, not @dnolen directly!
swlabrtyr 2017-06-09T08:35:10.022405Z

thanks a lot @danielstockton ! I’ll keep at it

2017-06-09T12:15:00.897976Z

Relating to my earlier question, I've noticed two almost identical join queries where one was being normalized and one wasn't. The difference was that the normalized result was a parameterized query. Sure enough, the other one was normalized when I modified the ast to include a random param. Can anyone suggest why that might be? {:join [:p1 :p2]} vs ({:join [:p1 :p2]} {:test :test}) with the parameters having no effect other than forcing through normalization.

2017-06-09T12:37:15.206726Z

Is there any reason not to use sablono instead of om/dom? Sablono seems to work ok (in my simple, beginner’s app) and is slightly easier to use, but all om tutorials use om/dom, so maybe there’s something I’m missing? I’d hate to hit some problem later and have to rewrite all back to om/dom…

2017-06-09T12:41:09.266304Z

@bbktsk I think the only thing to keep in mind when choosing Sablono at the moment is that it does not support server-side rendering

2017-06-09T12:42:35.288565Z

@symfrog Thanks, that’s definitely not an issue for me.

2017-06-09T14:43:18.781860Z

Has anyone else had problems with checkboxes since react was bumped?

dnolen 2017-06-09T17:00:12.884013Z

looking for feedback on the new README, it’s mostly just pairing the whole thing down and linking to om.next docs

dnolen 2017-06-09T17:00:15.885180Z

https://github.com/omcljs/om/tree/new-readme

1
anmonteiro 2017-06-09T17:33:24.575246Z

@dnolen can you open a PR so I can leave some inline comments?

dnolen 2017-06-09T17:34:03.588791Z

@anmonteiro absolutely, done!

anmonteiro 2017-06-09T17:34:30.598565Z

mostly just typos in the code examples

anmonteiro 2017-06-09T17:39:15.696886Z

@dnolen left some very minor comments

anmonteiro 2017-06-09T17:39:51.708728Z

a more general comment is that the README looks very slim currently, but we can improve on that / add more content later

dnolen 2017-06-09T17:41:08.734806Z

@anmonteiro yes

2017-06-09T18:01:32.172560Z

@dnolen om-cljs google group isn’t public readable without membership

dnolen 2017-06-09T18:01:52.180475Z

@dustingetz huh ok, thanks will look into it

julianwegkamp 2017-06-09T20:56:01.496070Z

@dnolen I think there should be a line explaining that the readme is about om.next to avoid confusion.