Deprecate typeKey and replace with modelName

[fivetanley]

modelName is a dasherized typeKey that we can be more rigorous about it being dasherized vs camelcased/snake cased/whatever.

This allows us to be more consistent in internals when accessing models via the container.

[stefanpenner]

When we move away from factories having concrete references to other factories and all API consume strings only. I think it is possible to stop polluting factories with a string name of any kind. This would finally let us fix the container and DI system which extends to shelter Ed from its factory polution

[tim-evans]

I'm using this in production code to DRY up some actions. I would love for this to be a deprecation notice before giving it the axe.

[igorT]

It will definitely be deprecated first.

[tim-evans]

@igorT thank ya! ❤️

[fivetanley]

closed by #3033