thinkthroughmath/dev_wizard

"Needs Code Review" removal removes PRs from my dashboard

Opened this issue · 0 comments

joelmccracken commented

Say Persons A and B are assigned to review a PR.

  1. Person A reviews PR.
  2. A finds an issue, removes "Needs CR" and adds "Needs Work"
  3. Person B looks at their dashboard.
  4. Dashboard for Person B is now missing PR from their dashboard.

Personally, I want to be able to see all the PRs to which I am assigned as reviewer, not just the ones that are currently marked "needs CR".

I propose changing that section of the dashboard so that it shows all PRs to which I am assigned, grouped by state -- "Needs Code Review" first, "Needs Work" second, and "Passed Code Review" third.