Yea, it seems weird to me to have a sub-resource controller where the index method for that sub-resource is not in it's controller file. +1 to move in a separate PR - card or issue to track and I'm satisfied :)