Fork me on GitHub
#untangled
<
2016-11-07
>
tony.kay20:11:45

just pushed an update to 0.6.0-SNAPSHOT. Includes a couple of little bug fixes

mitchelkuijpers21:11:20

Are you guys open for a pull request so I can set the shared property of the reconciler?

tony.kay21:11:50

@adambros Ooops. Yeah, the way return handler is written is wrong. My bad. We're in the middle of a swap at that time, so swapping is going to break.

gardnervickers21:11:31

@mitchelkuijpers We’ve had a lot of success putting shared state in the app db.

mitchelkuijpers21:11:58

@gardnervickers we have some global stuff like a certain base url and in which product we are (we are writing a plugin for multiple products) and now we just reference global values, but setting a shared value on the reconciler is way nicer. Then you can use om/shared in your components to reference them

mitchelkuijpers21:11:26

And it is pretty handy with devcards ^^

tony.kay21:11:42

@mitchelkuijpers sure, but I think DNolen deprecated that feature in favor of link queries

gardnervickers21:11:53

We had a similar setup, where there was some global state we wanted to read. Putting it under the root of the DB and just querying for it like [:my/thing ‘_] worked well, and I couldn’t figure out a case where shared makes more sense.

tony.kay21:11:35

though I seem to remember there was some use-case he was considering leaving it for

mitchelkuijpers21:11:40

Oh ok.. maybe not a good idea to add support for it then

tony.kay21:11:42

it's a trivial add, so feel free

mitchelkuijpers21:11:01

Also I had a question about using the new load to load a simple property from the server like [:foo] I currently still use load-data for this

mitchelkuijpers21:11:21

because of the required SubqueryClass

mitchelkuijpers21:11:10

@gardnervickers We mostly used it because of reasons 😛

anmonteiro21:11:15

:shared is not deprecated, and it serves different use cases than link queries

jdunruh21:11:21

«now 90 1 / 20 fno tI can’t call me call me.1 hyphen

mitchelkuijpers21:11:47

Then I will create a pull request 🙂 thnx @anmonteiro

tony.kay22:11:18

@mitchelkuijpers merged and I pushed a SNAPSHOT update to clojars

tony.kay22:11:33

Oh, and the load API. Hm, I guess in that sense load-data was more direct. How about we make the class optional?