Web UI display wrong PR status in case 'approve' status was change to 'inapproved'

[yafeldman]

Web UI display wrong PR status in case 'approve' status was change to 'inapproved'

[Levi-Lesches]

I think this depends on your branch settings: do you require a minimum amount of approvals? It does show "4 pending reviewers", presumably one of them is the self-request. But you also already have 2 approvals, which may be enough to merge.

[yafeldman]

We do require 2 approvals. But this issue is not about merging PR. There is discrepancies in UI: the top shows 1 approvals whereas bottom indicates 2 approvals. The approval from Sanjeevani19 was reverted from 'approve'

[Levi-Lesches]

Hm, I see what you're saying, but it actually looks like it's consistent: Sanjeevani19 approved, so that counts as an approval, then self-requested another review. What happens then is GitHub notes that they want to review it again, but doesn't remove the old approval. That's why "2 approvals" remains, and the top notes that they are due for another review. That's also why you see a yellow dot by their name on top instead of a green check on top, but both the top and the bottom agree that there are "4 pending reviewers". (Notice how the bottom shows 2 approvals + 4 pending = 6 total, but the top only shows 5)