Bug in aggregate subscriptions based on rum derived atoms ? about citrus HOT 10 CLOSED

Yeah. Going to talk to Nikita about this.

from citrus.

As a temporary workaround I'll do the following. What do you think ?

(let [rec   (scrum/reconciler {:state     (atom {})
                               :resolvers {[:a] (constantly :a)
                                           [:b] (constantly :b)}})
      sub-a (fn [reconciler]
              (scrum/subscription reconciler [:a]))
      sub-b (fn [reconciler]
              (scrum/subscription reconciler [:b]))
      derived (fn [reconciler]
                (let [a @(sub-a reconciler)
                      b @(sub-b reconciler)]
                  [a b]))]
  (derived rec))

The example is dumb, but I hope you get the point.

from citrus.

FYI, not working, since I'm using it as a subscription (with (rum/react XXX))... I think I'm blocked...

from citrus.

Hey. You are right. I'm going to update the code later today. Which version of Scrum is it?

from citrus.

2.1.0-SNAPSHOT. Sorry I should have mentionned it.

Going to upgrade to 2.2.0 to ease the transition to the bugfix version.

from citrus.

No worries, I'll update both.

from citrus.

Actually I think that's inconsistency in Rum. When used in Clojure derived-atom shouldn't add watchers.

from citrus.

Indeed, I think you're right : you render in one pass server-side, watches are useless. At least so it seems. Are you opening an issue in Rum ?

from citrus.

Fixed in 86d9adf for 2.1.0 and 2.2.9 in master. Please check if it works for you.

from citrus.

Looks like it works fine in 2.1.0. Thanks a lot !

from citrus.