Fork me on GitHub
#om
<
2017-05-15
>
dnolen11:05:31

@anmonteiro @xiongtx he’s actually referring to the the change I made for the defui macro

dnolen11:05:38

ha, acually sorry I think he just misremembered what I said and I forget I hadn’t made that change yet

dnolen11:05:14

there’s a bunch of annoying code in defui for parsing the macro body - it can be replaced with spec

dnolen11:05:57

we haven’t adopted spec yet because of the 1.9.0 alpha dep - but I think we’ll make the jump when 1.9.0 gets released

anmonteiro21:05:00

@peeja @matthavener proposal for a smart default in Compassus’s :send https://github.com/compassus/compassus/pull/30 Seeking feedback / thoughts if this might break any assumptions we currently have? pinging you 2 since you’re the most recent contributors

matthavener22:05:19

@anmonteiro seems legit to me. although, if the remote returns data in a slightly different format than the current query, normalization can do really confusing things

matthavener22:05:37

then again, there are a lot of confusing things for most om.next beginners 🙂

anmonteiro22:05:21

@matthavener sure, but my thought there was that if you’re changing the query you send to the remote, you’ll also pass it to the callback anyway

matthavener22:05:34

yeah, I think thats reasonable

matthavener22:05:48

@cmcfarlen probably has an opinion as well

anmonteiro22:05:53

this is just about establishing a sensible default that will not make users wonder why their data is not being normalized

anmonteiro22:05:35

thanks for your input