incubator-bloodhound-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Apache Bloodhound" <bloodhound-...@incubator.apache.org>
Subject Re: [Apache Bloodhound] #312: Configure ticket fields in create ticket shortcut
Date Wed, 19 Dec 2012 01:51:02 GMT
#312: Configure ticket fields in create ticket shortcut
--------------------------+------------------------
  Reporter:  olemis       |      Owner:  gjm
      Type:  enhancement  |     Status:  review
  Priority:  major        |  Milestone:
 Component:  ui design    |    Version:
Resolution:               |   Keywords:  ticket qct
--------------------------+------------------------

Comment (by rjollos):

 This is a redundant comment with what I said in #168, but I favor this
 change. I'd suggest an option that is more descriptive than
 `shorcut_fields` though. In my original patch for #168, I named it
 `quick_create_fields`. I think `quick_ticket_fields` is better though.

 Olemis: I think you may have intended to point to
 [/attachment/ticket/234/t234_r1423108_qct_desc_v2.diff this patch].

 If we allow this degree of configuration, we may want to go a step further
 remove the restriction on select fields eventually. What is the motivation
 for imposing that restriction at this time? We should mention in the doc
 string for the option that this restriction is in place, otherwise users
 might think it is a bug when they try to add a custom text field to the
 quick ticket form. This should be an easy change for the committer to
 make, no need to refresh the patch.

-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/312#comment:3>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound (incubating) issue tracker

Mime
View raw message