Fork me on GitHub
#omnext
<
2015-12-29
>
janherich13:12:22

Hi everybody ! I would like to ask one question about merging new state deltas into app-state. The format of the state delta which default-merge (https://github.com/omcljs/om/blob/master/src/main/om/next.cljs#L1348, argument named res) expects seems to be little bit odd: {some/mutation {:tempids {[:key-path tempid] [:key-path real-id]}}} is the correct form for the default-merge to resolve tempids. But it doesn't pick state delta when it's nested in {some/mutation {:result some-state-delta}} - so it's not possible to directly feed external (server) mutation parse result to merge! function. It's possible only when we are parsing read expression or mutation which results in tempids... It's not particularly difficult to handle, but it seems little bit strange to me, am i missing something ?