? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
23 May 2019

Pull Request for Issue # .

Summary of Changes

This adds a layout for items to remove duplicate code.

Testing Instructions

The module should work like before.

Documentation Changes Required

IDK.

avatar SharkyKZ SharkyKZ - open - 23 May 2019
avatar SharkyKZ SharkyKZ - change - 23 May 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 May 2019
Category Modules Front End
avatar franz-wohlkoenig franz-wohlkoenig - change - 23 May 2019
Title
[4.0][mod_articles_category] Add items sublayout
[4.0] [mod_articles_category] Add items sublayout
avatar franz-wohlkoenig franz-wohlkoenig - change - 23 May 2019
Title
[4.0][mod_articles_category] Add items sublayout
[4.0] [mod_articles_category] Add items sublayout
avatar franz-wohlkoenig franz-wohlkoenig - edited - 23 May 2019
avatar SharkyKZ SharkyKZ - change - 23 May 2019
Labels Added: ?
avatar Quy
Quy - comment - 25 May 2019

I have tested this item successfully on 84dbc89


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24991.

avatar Quy Quy - test_item - 25 May 2019 - Tested successfully
avatar joomla-cms-bot joomla-cms-bot - change - 5 Jun 2019
Category Modules Front End Unit Tests Administration com_admin com_associations com_banners com_cache com_categories com_config com_contact
avatar SharkyKZ SharkyKZ - change - 5 Jun 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 5 Jun 2019
Category Unit Tests Administration com_admin com_associations com_banners com_cache com_categories com_config com_contact Modules Front End
avatar SharkyKZ SharkyKZ - change - 5 Jun 2019
Labels Removed: ?
avatar jwaisner jwaisner - test_item - 30 Jan 2020 - Tested successfully
avatar jwaisner
jwaisner - comment - 30 Jan 2020

I have tested this item successfully on 52c8656


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24991.

avatar jwaisner
jwaisner - comment - 30 Jan 2020

@Quy Please retest.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24991.

avatar Quy Quy - test_item - 5 Feb 2020 - Tested successfully
avatar Quy
Quy - comment - 5 Feb 2020

I have tested this item successfully on 52c8656


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24991.

avatar Quy Quy - change - 5 Feb 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 5 Feb 2020

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24991.

avatar richard67
richard67 - comment - 9 Feb 2020

Restarted drone.

avatar richard67
richard67 - comment - 9 Feb 2020

@SharkyKZ I can't see conflicts for this PR here on GitHub, but the clone step in drone reports merge conflicts when fetching your PR into a 4.0-dev. Could you check if you can update this PR to current 4.0-dev and solve conflicts if you have some? If not I could try to have a look, ping me then.

avatar SharkyKZ SharkyKZ - change - 9 Feb 2020
Labels Added: ?
avatar SharkyKZ
SharkyKZ - comment - 9 Feb 2020

Branch updated. Didn't see any conflicts.

avatar richard67
richard67 - comment - 9 Feb 2020

Then good luck with the drone .. maybe it just has severe hickup at the moment ;-)

avatar richard67
richard67 - comment - 9 Feb 2020

Clone step already passed in drone .. that's a good sign.

avatar richard67
richard67 - comment - 9 Feb 2020

@SharkyKZ Now GitHub shows me "This branch cannot be rebased due to conflicts" below all the green passed checks.

avatar SharkyKZ
SharkyKZ - comment - 9 Feb 2020

@richard67 What about now?

avatar richard67
richard67 - comment - 9 Feb 2020

Same:
pr-24991
@wilsonge What does that mean? I don't see any conflicts.

avatar Quy
Quy - comment - 9 Feb 2020

Strange. I am not getting the message you got. I get the green button to squash and merge.

avatar richard67
richard67 - comment - 9 Feb 2020

@Quy I see. I had rebase and merge selected from previous merge. Then it shows the above. Now I have changed to squash and merge in the dropdown and see what you say. Got it. In some case rebase and merge isn't possible, George told me. All ok.

avatar richard67
richard67 - comment - 9 Feb 2020

Question now is shall we wait for the outstanding review by the code owner? @Quy What do you think?

@SharkyKZ Sorry for unnecessary rumor ;-)

avatar Quy
Quy - comment - 9 Feb 2020

Oops I have not been paying attention to Reviewers. I don’t know what the rules are so I guess we wait.

avatar richard67
richard67 - comment - 9 Feb 2020

I have pinged him here and on Glip, let's see what happens.

avatar richard67
richard67 - comment - 9 Feb 2020

I have tested it with success, too, and review looks good to me, so I merge.

avatar richard67 richard67 - close - 9 Feb 2020
avatar richard67 richard67 - merge - 9 Feb 2020
avatar richard67 richard67 - change - 9 Feb 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-02-09 16:51:49
Closed_By richard67
avatar richard67
richard67 - comment - 9 Feb 2020

Thanks.

Add a Comment

Login with GitHub to post a comment