Fixes the logic used for what we consider a scoring pull request in the user. When a user wins, we store a frozen receipt on their winning PRs. This has now been updated to just freeze their first four eligible PRs, so they will always display on the profile. No other PRs are frozen and will show based on their current state on GitHub.
Test process
Register with a user
Have PRs such that there is at least one waiting and one invalid PR intermixed with four other waiting PRs
Ensure the profile renders all the PRs in the expected state
Set four of the waiting PRs to be eligible in the database
Reload the profile and observe that the four eligible PRs are hoisted to the top
The remaining waiting and invalid PR are shown below
Ensure that the four eligible PRs have been stored in the user receipt in the database
Requirements to merge
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code in hard-to-understand areas
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
Description
Fixes the logic used for what we consider a scoring pull request in the user. When a user wins, we store a frozen receipt on their winning PRs. This has now been updated to just freeze their first four eligible PRs, so they will always display on the profile. No other PRs are frozen and will show based on their current state on GitHub.
Test process
Requirements to merge