Rename has_approved? methods to approved_by?

What does this MR do?

Mostly addressing !36564 (comment 379170457). Better naming for a method which should mean merge request approved by a user?