#7147 closed enhancement (fixed)
Plugin Repo: Sort assigned reviews by date
Reported by: | mrfoxtalbot | Owned by: | dd32 |
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Plugin Directory | Keywords: | |
Cc: |
Description
The Assign plugins to reviewers feature has been extremely useful. As it turns out, the way the feature works at the moment, it does not allow to sort reviews by when they were assigned.
As team members have more and more reviews marked as pending, it would be extremely useful to be able to sort them by the date they were assigned. That way it would be easier to browse thought your own reviews chronologically.
I was thinking about a new sortable column labeled "Assigned Date" but that might not be the best option.
Change History (4)
#2
follow-up:
↓ 3
@
17 months ago
Perhaps just marking Assigned Reviewer column as sortable, and having that sort by the time would work?
I think this is the best approach, yes.
Would we ever want the ability to sort it by assigned human?
Good question... Personally, I do not think this is necessary but let's ask those using this feature more actively. What do you think @frantorres, @pacomarchante, @eherman24?
#3
in reply to:
↑ 2
@
17 months ago
Perhaps just marking Assigned Reviewer column as sortable, and having that sort by the time would work?
That sounds perfect to me :D
The idea is that if you filter by "Assigned Reviewer (Person)" and sort by "Assigned Reviewer (Time)" you are able to see the last reviews from that person.
Would we ever want the ability to sort it by assigned human?
I don't think it would be necessary to sort by human, so we can make use that column for sorting by the time.
I was trying to avoid having too many columns, which is why the assigned date is currently part of the assignee column.
Perhaps just marking
Assigned Reviewer
column as sortable, and having that sort by the time would work?Would we ever want the ability to sort it by assigned human?
Alvaro (1hr ago), Alvaro (2days ago), Alvaro (3 days ago), Dion (2h ago), Dion (5h ago)
Alvaro (1h ago), Dion (2h ago), Dion (5hr ago), Alvaro (2days ago), Alvaro (3 days ago)