This feature seems to have broken some custom JS that is in the
email-ext plugin. In the config.jelly it has a couple script blocks.
One near the top looks like this:
<script...>var extEmailInit = new Array();</script>
Then later this variable is used
<script...>if(extEmailInit['...'] != null) { }</script>
In the second code block, extEmailInit is not defined and so throws an
error in the JS console of the browser. How do I update the plugin to
still support this custom JS block and have it work correctly?
Thanks,
slide
On Mon, May 14, 2012 at 3:50 PM, Paul Eipper <lkrai...@gmail.com> wrote:
> Very good feature. I also vote to remove the multiple plugins restriction,
> at least Publish Over SSH even lists that as a way to perform multiple
> commands on its help:
> "If you want to Exec before the files are transferred, use 2 Transfer Sets
> and move the Exec command before the Transfer set that includes a Source
> files pattern."
>
> --
> Paul Eipper
>
> On Sunday, May 13, 2012 2:37:14 PM UTC-3, domi wrote:
>>
>> Ulli, thats a really good idea!!!!
>> I think the sam would actually be helpful for triggers, these are getting
>> more and more too…
>> Domi
>>
>> On 13.05.2012, at 19:18, Ulli Hafner wrote:
>>
>> While I think that reordering and showing only active plug-ins is a great
>> improvement I think the selection of the action still is not optimal. Using
>> a List Box Button to select publishers is not very user friendly (at least
>> if you have more than 10 available). Especially when you need to scroll to
>> the bottom of the list it needs quite a lot of time until you reach the last
>> element of the list (maybe the scrolling can be improved anyway). The same
>> pattern is currently used for the portlets selection in the dashboard view
>> and I think it could be also improved by using a more intuitive selection
>> "view". E.g., something in the style of the Jira dashboard selection. In
>> Jira when you press the add button on the dashboard then a selection dialog
>> is opened where you see small previews of the available portlets. This would
>> also make sense for the publishers: here we can show a small icon, a
>> description, etc. Maybe we can also groupd by category...
>>
>> Ulli
>>
>>
>> On 05/11/2012 12:40 PM, Peter Franzén wrote
>>
>> Hi,
>>
>> A great improvement, but some Plugins are not added when I select them.
>> I tried some and works perfectly, but I can not add:
>> Coverity
>> Editable Email Notification
>>
>> Just nothing happens when I select them
>>
>> /Peter
>>
>>
>> Den torsdagen den 19:e april 2012 kl. 18:01:26 UTC+2 skrev Kohsuke
>> Kawaguchi:
>>>
>>> Following up the conversation from JUC Paris, I pushed the "publisher"
>>> branch to the repository that changes the publisher configuration UI.
>>> I'd like to get feedback on whether people think this is a good idea
>>> or a bad idea.
>>>
>>> Currently, the "post-build action" of the project configuration page
>>> lists one checkbox per one publisher. This has a couple of problems:
>>>
>>> - when you have a large number of plugins, as many large instances
>>> inevitably do, they occupy a large real estate in the configuration
>>> screen.
>>>
>>> - you cannot reorder them
>>>
>>> In the branch, instead of a big list of checkboxes, you get a
>>> drop-down menu button just like the main "build' section. And then
>>> you'll be adding publishers by clicking them.
>>>
>>> - once you add a publisher of a certain kind, it gets grayed out and
>>> prevents you from adding 2nd (and if you actually click the grayed out
>>> item, you jump to the corresponding part of the page.) (this
>>> restriction of one instance per one publisher might be something worth
>>> removing in the future?)
>>>
>>> - new items are inserted into their preferred position, instead of
>>> getting added to the bottom, so that users normally wouldn't have to
>>> think about the proper ordering among them (you generally want the
>>> data collection tasks to come before the notification tasks.)
>>>
>>> - a few animations to go with it.
>>>
>>> Looking forward to hearing back.
>>>
>>> --
>>> Kohsuke Kawaguchi
>>
>>
>>
>>
>
--
Website: http://earl-of-code.com
