I believe we used to have the types in the ids module, and @mhrivnak switched to using the models for the types. I think keeping the types on the models is more natural than keeping them in the ids module, so I think we should keep this the way it was.