? Pending

User tests: Successful: Unsuccessful:

avatar fcoulter
fcoulter
28 Sep 2016

Pull Request for Issue # .

Summary of Changes

removes commented out code, it just looks messy having it in a production release

Testing Instructions

should not require testing, because removed code is not executable.

Documentation Changes Required

removes commented out code, just tidying up

avatar fcoulter fcoulter - open - 28 Sep 2016
avatar fcoulter fcoulter - change - 28 Sep 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Sep 2016
Category Administration Components
avatar joomla-cms-bot joomla-cms-bot - change - 28 Sep 2016
Labels Added: ?
avatar zero-24
zero-24 - comment - 28 Sep 2016

Can you solve the merge conflicts or redo the PR against the current staging?

This can help you if you want to redo this against staging using the GitHub Web UI https://docs.joomla.org/Using_the_Github_UI_to_Make_Pull_Requests

avatar fcoulter fcoulter - change - 29 Sep 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-09-29 16:29:12
Closed_By fcoulter
avatar fcoulter fcoulter - close - 29 Sep 2016

Add a Comment

Login with GitHub to post a comment