When I look at this I don't understand what `@groups` is without looking at the controller. In general I think we wanna stay away from defining extra instance variables in the controller. Instead I think we should add a scope to user, e.g. `@user.public_goups`