MM-813: Fixed the issue of wrong name shown in subscription when PR is moved from draft to ready for review

[Kshitij-Katiyar]

Description

Fixed the issue of the wrong name shown in the subscription when PR is moved from draft to ready for review

Ticket link

Fixes #813

What to test

• Create a subscription for a repo
• Create a PR with user B and mark it as draft
• Mark the PR as ready for review from user A

Now

#new-pull-request by B

Before

#new-pull-request by A

[AayushChaudhary0001]

@Kshitij-Katiyar I was testing this PR and this functionality is not working, can you please take a look at it as discussed?
Adding screenshots for reference:-

[Kshitij-Katiyar]

@Kshitij-Katiyar I was testing this PR and this functionality is not working, can you please take a look at it as discussed? Adding screenshots for reference:-

@AayushChaudhary0001 I think the PR is working as expected, this PR aims to fix the issue that when a user moves a PR from draft to ready for review, his name is shown in the subscription message that he created a new PR regardless of the author of the PR.
With the changes of this PR, no matter who moves the PR from draft to ready to review, it will show the author of the PR in the subscription message.

[AayushChaudhary0001]

This PR has been tested according to the above clarification, and it is working fine. The user receives notification of the user who created the PR despite who moved it to another state, LGTM. Approved.