Add the ability to sort the reviews result in the "Reviews" page on any column

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 2.9
  • Component/s: Manual Review
  • Labels:
    None

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Fabrice Bellingard added a comment -

The sort works on every column except the "severity" one, which only groups reviews by severity (but the order does not reflect the severity order).

Show
Fabrice Bellingard added a comment - The sort works on every column except the "severity" one, which only groups reviews by severity (but the order does not reflect the severity order).
Hide
Permalink
Freddy Mallet added a comment -

Sorting on "Project" and "Assignee" don't work with Derby. You can simply drop the possibility to sort those columns.

Show
Freddy Mallet added a comment - Sorting on "Project" and "Assignee" don't work with Derby. You can simply drop the possibility to sort those columns.
Hide
Permalink
Fabrice Bellingard added a comment -

Sort removed on the Assignee and Project columns, as the DB request fails on Derby.
(FI: it was OK on Postgres)

Show
Fabrice Bellingard added a comment - Sort removed on the Assignee and Project columns, as the DB request fails on Derby. (FI: it was OK on Postgres)
Hide
Permalink
Freddy Mallet added a comment -

Tested

Show
Freddy Mallet added a comment - Tested
Hide
Permalink
Fabrice Bellingard added a comment -

While writing ITs, I found a regression: it was not possible to unassign a review. I fixed it.
Freddy, if you want to test, feel free. Otherwise, you can directly reassigned it to me so that I proceed with ITs.

Show
Fabrice Bellingard added a comment - While writing ITs, I found a regression: it was not possible to unassign a review. I fixed it. Freddy, if you want to test, feel free. Otherwise, you can directly reassigned it to me so that I proceed with ITs.
Hide
Permalink
Freddy Mallet added a comment -

Tested !

Show
Freddy Mallet added a comment - Tested !

People

  • Assignee:
    Fabrice Bellingard
    Reporter:
    Freddy Mallet
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: