If a Phabricator review request is still pending, but another reviewer has Approved, the review isn't included in the total

[mikeconley]

There's a limitation in the Differential revision query API where review requests are not listed in the following scenario:

1. The user is one of several reviewers on a revision
2. The user is a non-blocking reviewer
3. Another user has given an approval such that the revision gets the Approved status

This is something that mcote's team has been banging against too.

It might be simpler to get an accurate total of Phabricator reviews by just scraping the dashboard.

[mikeconley]

Another thing that can occur is that if a user is one of several blocking reviewers, even if that user has completed a review on the latest revision, the review request will still be set at the "needs-review" state, so it'll keep showing up in the query.

[mikeconley]

This should be fixed in 0.2, which uses page scraping instead of the Conduit API.