Yes it's a TODO. Basically we're hardcoding model.id in some places, we should use idAttribute which may take the form of a function or a string. I plan to do it later and add specific unit tests for it (that's why I kind of skipped it for now)