Utilise 'needs:' for review-qa-smoke

What does this MR do?

Since #31264 (closed) got resolved, we can now utilise needs: for review-qa-job to depend on review-deploy. It will help to avoid failures when Review App is not deployed successfully but smoke tests are still attempted.

See #323999 (closed) & gitlab-com/www-gitlab-com#11568 (closed) for more information.

Does this MR meet the acceptance criteria?

Conformity

• 📋 Does this MR need a changelog?
  • I have included a changelog entry.
  • I have not included a changelog entry because _____.
• Documentation (if required)
• Code review guidelines
• Merge request performance guidelines
• Style guides
• Database guides
• Separation of EE specific content