avatar roland-d
roland-d
31 May 2014

In the old tracker the search field would also search for the pull ID, now it only searches the Joomlacode issue ID. It would be better to search on both as we often look for the pull ID.

An example is searching for pull ID 3362, it only shows Joomlacode issue IDs with this combination in it.

avatar roland-d roland-d - open - 31 May 2014
avatar mbabker
mbabker - comment - 31 May 2014

With the current dev head, the search filter works in two ways:

  1. A plain search queries based on title
  2. Prefixing a search with id: searches based on the pull ID

What do you suggest?

avatar roland-d
roland-d - comment - 31 May 2014

I see now what the difference is between the old and current dev of patchtester. The old patchtester has search field for Pull ID and a search field for Title. Can't we just keep those?

It is clear for users what they are searching for and the Joomlacode issue ID will be removed also once we drop the Joomlacode tracker. Having to use id: is not very userfriendly in my opinion.

So my preference would be to keep the search filters as they are now.

avatar mbabker mbabker - close - 31 May 2014
avatar mbabker
mbabker - comment - 31 May 2014

Part of what I was aiming for was standardizing the interface on the "CMS standard" which is why I consolidated the search filters into one field.

I've changed the search behavior so that it'll return matches to the pull request title, pull request ID, or JoomlaCode ID without any identifiers.

avatar roland-d
roland-d - comment - 31 May 2014

Tested the change and it works perfect this way ?

Add a Comment

Login with GitHub to post a comment