fixing #8491 - removing puppet modules list from content view version api

This takes the CVV index api from ~5 minutes down to 17 seconds
for 47 content view versions, 10 having 100 puppet modules, the rest having
5
> I'm wondering if the better solution is to use foreman's model of having a shared template (main) that the show and index templates include. See operating systems for example
 
Big :+1: to this. Having standard templates for resources makes this a lot easier, not even just index/show, but things like this too https://github.com/Katello/katello/pull/4831