Success

User tests: Successful: Unsuccessful:

avatar b2z
b2z
13 Mar 2016

Pull Request for Issue # 723.

Summary of Changes

Added milestones filter to issues filter bar and moved the filter bar to separate template.

Testing Instructions

Check that milestones filter is working correctly and other filters did not break.
You can test on my testing server.

avatar b2z b2z - open - 13 Mar 2016
avatar zero-24
zero-24 - comment - 13 Mar 2016

:+1: Looks good here :)

avatar mbabker
mbabker - comment - 13 Mar 2016

My one suggestion would be to sort the milestones in order, otherwise looks fine.

avatar b2z
b2z - comment - 14 Mar 2016

@mbabker currently getMilestones() is ordered by milestone_number. What do you suggest?

avatar mbabker
mbabker - comment - 14 Mar 2016

The title if practical. One of the screens I looked at the filters were in
a "funny" order (not quite this but it felt something like 3.3.6, 2.5.28,
3.4.0, 3.5.0, 3.4.1)

On Monday, March 14, 2016, Dmitry Rekun notifications@github.com wrote:

@mbabker https://github.com/mbabker currently getMilestones()
https://github.com/joomla/jissues/blob/master/src/App/Projects/TrackerProject.php#L365
is ordered by milestone_number. What do you suggest?


Reply to this email directly or view it on GitHub
#757 (comment).

avatar b2z
b2z - comment - 14 Mar 2016

Updated - now milestones are ordered by title ;)

avatar mbabker mbabker - change - 14 Mar 2016
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-03-14 17:58:52
Closed_By mbabker
avatar mbabker mbabker - close - 14 Mar 2016
avatar mbabker mbabker - merge - 14 Mar 2016
avatar mbabker mbabker - reference | 913b476 - 14 Mar 16
avatar mbabker mbabker - merge - 14 Mar 2016
avatar mbabker mbabker - close - 14 Mar 2016
avatar mbabker mbabker - head_ref_deleted - 14 Mar 2016
avatar mbabker
mbabker - comment - 14 Mar 2016

It still puts the open milestones before the closed ones. But honestly I'm OK with this.

avatar b2z
b2z - comment - 14 Mar 2016

We could add order by state DESC ?

avatar mbabker
mbabker - comment - 14 Mar 2016

Nah, it's fine how it is now.

Add a Comment

Login with GitHub to post a comment