to make it more readable, I think this block can be packaged in another private function, so we could get something like

~~~ruby
clean_not_accepted_users.map do |user|
  ...
end

def clean_not_accepted_users
  ...
end
~~~

Another suggestion (perhaps for much later) is to move the logic of getting `clean` objects into another class (separate from the controller and model), I don't think this belongs in the controller