? Success

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
29 Apr 2016

This PR removes developers debug output that should never have been committed in the first place.

avatar PhilETaylor PhilETaylor - open - 29 Apr 2016
avatar PhilETaylor PhilETaylor - change - 29 Apr 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Apr 2016
Labels Added: ?
avatar Kubik-Rubik Kubik-Rubik - change - 29 Apr 2016
Milestone Added:
avatar SniperSister
SniperSister - comment - 29 Apr 2016

+1 from me for this, if one REALLY needs to debug this, a temporary debug statement does a perfect job

avatar Kubik-Rubik
Kubik-Rubik - comment - 29 Apr 2016

I also agree on this. I will merge on code review! Thank you @PhilETaylor and @SniperSister.

avatar Kubik-Rubik Kubik-Rubik - close - 29 Apr 2016
avatar Kubik-Rubik Kubik-Rubik - merge - 29 Apr 2016
avatar Kubik-Rubik Kubik-Rubik - reference | 6160ff0 - 29 Apr 16
avatar Kubik-Rubik Kubik-Rubik - merge - 29 Apr 2016
avatar Kubik-Rubik Kubik-Rubik - change - 29 Apr 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-04-29 16:04:37
Closed_By Kubik-Rubik
avatar Kubik-Rubik Kubik-Rubik - close - 29 Apr 2016
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:
avatar PhilETaylor
PhilETaylor - comment - 19 Oct 2016

tagging @brianteeman as apparently he was the initial reporter

Add a Comment

Login with GitHub to post a comment