show the PRs that a user has reviewed on their profile - similar to the show PRs on user profile

[DonnieBLT]

No description provided.

[tsu-ki]

/assign

[github-actions]

Hello @tsu-ki! You've been assigned to OWASP-BLT/BLT issue #3283. You have 24 hours to complete a pull request.

[tsu-ki]

@DonnieBLT sir, how would we determine if a user has made reviews on a PR?

[DonnieBLT]

from the requested_reviewers field

[tsu-ki]

from the requested_reviewers field

understood, thank you

[iitzIrFan]

@tsu-ki Actually you will need an model to get the required data which I and @DonnieBLT were working on and were waiting for the model pr to be reviewed and to be merged in main branch !

[tsu-ki]

@tsu-ki Actually you will need an model to get the required data which I and @DonnieBLT were working on and were waiting for the model pr to be reviewed and to be merged in main branch !

another model isn't necessarily required. In the get_context_data method of UserProfileDetailView, after getting the user's github url, I'm extracting their GitHub username from the github_url. Then, using the GitHub API to fetch the reviewed PRs.

[iitzIrFan]

@tsu-ki Nice approach actually Donnie mentioned the models pr as an reference so got this approach at that, time

[DonnieBLT]

We will also want to have the model to do the leaderboard

[tsu-ki]

We will also want to have the model to do the leaderboard

sir, so we have to store the reviewer information in DB?

[DonnieBLT]

Yes, for reviewers and contributors

[iitzIrFan]

@DonnieBLT This is there in the model pr I guess?

[DonnieBLT]

Yes https://github.com/OWASP-BLT/BLT/pull/3266/files

[iitzIrFan]

@DonnieBLT It's still in review process so @tsu-ki need to wait for some time also I want some issues to work on @DonnieBLT can you help me out!

[tsu-ki]

Yes https://github.com/OWASP-BLT/BLT/pull/3266/files

@DonnieBLT sir, should I wait for the corresponding PR to be merged?

[DonnieBLT]

You could copy the code and add it here. I’ll make sure to merge both branches so you both get credit.

[tsu-ki]

You could copy the code and add it here. I’ll make sure to merge both branches so you both get credit.

sure, I'll rebase my branch and work on the new model created by @SahilDhillon21

[SahilDhillon21]

Apologies for the delay, I'll push the code by tomorrow

[github-actions]

⏰ This issue has been automatically unassigned due to 24 hours of inactivity.
The issue is now available for anyone to work on again.