component.onModelChange ?

[tsjoberg]

Believe @markijbema can shed some light on this but I couldn't find an answer:

https://github.com/clayallsopp/react.backbone/blob/master/react.backbone.js#L37

is component.onModelChange a React component property? or added by react.backbone mixin? I couldn't find any reference to this in React docs or in the mixin code as well

[markijbema]

Seems like an extension point. Default is force update, but if you override onModelChange, that's called instead. If this isn't in the docs, lets keep this issue open until it is ;)

[tsjoberg]

Just a proposal but would it make sense to just remove this altogether? Doesn't seem as useful and is not documented anywhere so people would not know this existed without checking the code?

[markijbema]

It has been here since the very first commit. If you want to remove it, I think we should at least add a deprecation warning:
c15403e

@clayallsopp do you remember why you added this? Do you use this?

[clayallsopp]

I don't use this method anymore, fine with removing it or deprecating