A talk (partly) about re-frame at the react conf: http://conf.reactjs.com/schedule.html

Nice! Will watch

Hey guys, I have a weird bug, I'm under the impression re-frame/reagent does not update the ui for a component parameter change:

(defn base [id]
  (let [show-db-summary (subscribe :show-db-summary?)]
    (fn []
      [:div#main {:className "sidebar-l sidebar-o side-scroll header-navbar-fixed"}
       [side-overlay]
       [sidebar]
       [header]
       (if @show-db-summary
         [db-summary]
         (case id
           :a [a/main]
           :b [b/main]
           (do (log/error "Unknown module:" id)
               (dispatch :navigate :404))))
       [footer]])))


(defn main []
  (let [module (subscribe :active-module)]
    (fn []
      (cond
        (= @module :welcome) [welcome/main]
        (contains? #{:a :b} @module) [base @module]
        :else [module-404]))))

when I switch between :a and :b, the [base id] is not updated

the rest of the system works fine, I display the :active-module in the header, it's updated correctly

Am I missing something?

(it renders correctly when figwheel reloads)

@lsenta: I didn't see all the details, perhaps there's something else missing, but one thing is that you need to put id in the args list of the inner fn in your base component. So it would be:

(defn base [id]
  (let [show-db-summary ...]
    (fn [id]
      ...)))

See the rookie mistake section at the end of https://github.com/Day8/re-frame/wiki/Creating-Reagent-Components#form-2--a-function-returning-a-function 😄

@lsenta: nberger has identified the bug, but that codes is not very re-frame-y

the way that it does an imperative (dispatch :navigate :404)

That is an attempt to change the state of the app. That decision should be in an event handler.

Views just render the current state

(and dispatch in response to external events) They don't make "decisions about state changes"