bug
avatar zero-24
zero-24
14 Aug 2014

Steps to reproduce the issue

See here: http://issues.joomla.org/tracker/joomla-cms/4108 or here http://issues.joomla.org/tracker/joomla-cms/4107

Expected result

The jissues-bot don't change the "local" status.

Actual result

The jissues-bot change the "local" status back to open if he add the labels.

avatar zero-24 zero-24 - open - 14 Aug 2014
avatar elkuku
elkuku - comment - 14 Aug 2014

Nasty...
I guess that this might be the hooks?
Maybe we can just leave the default off here: https://github.com/joomla/jissues/blob/master/src/App/Tracker/Controller/Hooks/ReceiveIssuesHook.php#L225 ?
Not sure if the hook data always has an action property?
@mbabker is the "Man of the Hooks™"

avatar elkuku elkuku - change - 14 Aug 2014
Labels Added: bug
avatar b2z
b2z - comment - 15 Aug 2014

Maybe we can just leave the default off here:

Agree.

avatar javigomez
javigomez - comment - 22 Aug 2014

The thing is that there is a lot of messages of activity that makes the view of an issue not very clean. See:
http://issues.joomla.org/tracker/joomla-cms/4144

This one doesn't give almost information:
screen shot 2014-08-22 at 11 04 46

While this is more related to this topic:
screen shot 2014-08-22 at 11 05 03

I think there is two kind of information in the issue: comment and activity (open, close...)

I think that the activities in the issue should be removed from the coments, or formatted so they look smaller and different, just like Github does:

screen shot 2014-08-22 at 11 06 32

Just my 2 cents ^_^

avatar b2z
b2z - comment - 22 Aug 2014

Good catch @javigomez - I think we need to create a separate issue for this improvement.

avatar zero-24
zero-24 - comment - 31 Aug 2014

Maybe we can just leave the default off here
[...]
Not sure if the hook data always has an action property?
@mbabker is the "Man of the Hooks™"

@mbabker Can we solve this by removing the default?

This comment was created with the J!Tracker Application at http://issues.joomla.org/.

avatar mbabker
mbabker - comment - 31 Aug 2014

I changed it, let's see what happens.

avatar zero-24
zero-24 - comment - 31 Aug 2014

thanks @mbabker

avatar zero-24
zero-24 - comment - 31 Aug 2014
avatar zero-24 zero-24 - change - 31 Aug 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-08-31 21:00:23
avatar zero-24 zero-24 - close - 31 Aug 2014

Add a Comment

Login with GitHub to post a comment