Add option to hide already-reviewed PRs

Issue #438 open
Ian Chamberlain created an issue

Currently, there is a predefined filter for Bitbucket PRs called "Pull requests to be reviewed" which is quite useful. However, it has a disadvantage of showing PRs that the user has already visited and completed review of.

I would like to propose either of two things:

  • A new filter "Unreviewed pull requests" which displays only pull requests for which the user has not set a review status (e.g. "Needs Work", "Approved")
  • A configuration option (possibly something like atlascode.bitbucket.explorer.hideReviewedPullRequests) which hides/shows pull requests in the explorer that have already had a review status set.

I have a somewhat related request to allow setting a default filter to show at startup, but I think I will open a separate request for that.

Comments (5)

  1. Ian Chamberlain Account Deactivated reporter

    As a note, it seems like these reviewed PRs already filter to the bottom when viewing all PRs, which is nice, but they do not move when using the "To be reviewed" filter. Sorting by unreviewed-first would be helpful here as well.

    Perhaps another alternative would be to add a status icon to the PR title showing that it has already been reviewed – currently the user must click into the PR and look at the approval button to figure out whether it was reviewed or not.

  2. Michael Blair

    There is something similar to this in the Bitbucket cloud dashboard Pull Requests list. There is a checkbox for “Needs my review” which will hide any pull requests already reviewed when checked. Something similar in VSCode would be very helpful.

  3. Ian Chamberlain Account Deactivated reporter

    Agreed, I eventually decided to implement it myself, so maybe this issue can be closed now: https://bitbucket.org/atlassianlabs/atlascode/pull-requests/972 and it was merged to main and released.

    The missing piece for me was setting it as the default filter, since otherwise I would have to change it every time I opened the PR view in VSCode. I implemented this change as well, but haven't gotten it merged yet: https://bitbucket.org/atlassianlabs/atlascode/pull-requests/992

    I have been using it for a few months with pretty good success, not sure what is needed for it to be merged back into the main extension for a new release.

  4. Log in to comment