Adds check for temporarily unapproved state
What does this MR do and why?
Since approvals can be asynchronously deleted, and we haven't yet patched #approved? to account for this, add this check so that we're not reporting an MR as approved when it is in an unapproved state.
Changelog: fixed
Related to #429470 (closed)