Consider whether store.push/pushPayload should copy

[igorT]

No description provided.

[aldhsu]

Not sure if this is a related issue. I think it would make sense if store.pushPayload returned the models pushed into the store.

At the moment on master link there is no return statement and store.pushPayload returns undefined.

Contrast to store.push, link, where the model pushed in is returned.

[wecc]

@aldhsu There's a feature named ds-pushpayload-return that enables pushPayload to return the result.

[pangratz]

@aldhsu also see RFC #161

[aldhsu]

@wecc @pangratz ok thanks!

[runspired]

We could probably do this in dev for safety but seems like unnecessary work overall.

[runspired]

as I've thought on this more, this is a Cache detail. Currently the cache does not retain this object (it copies, though by artifact and not design as it deconstructs resources into more granular caches) so it doesn't matter, if a Cache did retain this object it would likely want to deep-freeze it in dev as opposed to copying.