Making WordPress.org

Opened 9 months ago

Closed 5 months ago

#7147 closed enhancement (fixed)

Plugin Repo: Sort assigned reviews by date

Reported by: mrfoxtalbot's profile mrfoxtalbot Owned by: dd32's profile 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)

#1 @dd32
9 months ago

I was thinking about a new sortable column labeled "Assigned Date" but that might not be the best option.

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?

Sorted by Human Alvaro (1hr ago), Alvaro (2days ago), Alvaro (3 days ago), Dion (2h ago), Dion (5h ago)
Sorted by Assigned Date Alvaro (1h ago), Dion (2h ago), Dion (5hr ago), Alvaro (2days ago), Alvaro (3 days ago)

#2 follow-up: @mrfoxtalbot
9 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 @frantorres
9 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.

#4 @dd32
5 months ago

  • Owner set to dd32
  • Resolution set to fixed
  • Status changed from new to closed

In 13017:

Plugin Directory: Reviewer tools: Make the Reviewer column sortable.

Fixes #7147.

Note: See TracTickets for help on using tickets.