untangled

NEW CHANNEL: #fulcro
tony.kay 2016-11-07T20:39:45.003948Z

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

1🎉
mitchelkuijpers 2016-11-07T21:03:20.003950Z

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

tony.kay 2016-11-07T21:06:50.003951Z

@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.

gardnervickers 2016-11-07T21:07:31.003952Z

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

mitchelkuijpers 2016-11-07T21:09:58.003953Z

@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

mitchelkuijpers 2016-11-07T21:10:26.003954Z

And it is pretty handy with devcards ^^

tony.kay 2016-11-07T21:11:42.003955Z

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

gardnervickers 2016-11-07T21:11:53.003956Z

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.kay 2016-11-07T21:12:35.003957Z

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

mitchelkuijpers 2016-11-07T21:12:40.003958Z

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

tony.kay 2016-11-07T21:12:42.003959Z

it's a trivial add, so feel free

mitchelkuijpers 2016-11-07T21:15:01.003960Z

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

mitchelkuijpers 2016-11-07T21:16:21.003961Z

because of the required SubqueryClass

mitchelkuijpers 2016-11-07T21:18:10.003962Z

@gardnervickers We mostly used it because of reasons 😛

anmonteiro 2016-11-07T21:21:15.003963Z

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

2016-11-07T21:21:21.003964Z

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

mitchelkuijpers 2016-11-07T21:23:47.003965Z

Then I will create a pull request 🙂 thnx @anmonteiro

tony.kay 2016-11-07T22:20:18.003968Z

@mitchelkuijpers merged and I pushed a SNAPSHOT update to clojars

tony.kay 2016-11-07T22:26:33.003969Z

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