? Pending

User tests: Successful: Unsuccessful:

avatar dgrammatiko
dgrammatiko
1 Aug 2018

Pull Request for Issue # .

Summary of Changes

Converted the scripts for com_config

Testing Instructions

Go to http://joomla.local/administrator/index.php?option=com_config Everything works (first tab is selected)

Go to front end, Log in
Try to edit a module, everything still works

Goto /index.php/en/working-on-your-site/site-settings?view=config Everyhting still works (apply some changes and save)

Goto /index.php/en/working-on-your-site/template-settings?view=templates Everything still works (apply some changes and save)

Expected result

Actual result

Documentation Changes Required

No

avatar dgrammatiko dgrammatiko - open - 1 Aug 2018
avatar dgrammatiko dgrammatiko - change - 1 Aug 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Aug 2018
Category JavaScript Repository
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 1 Aug 2018 - Tested unsuccessfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 1 Aug 2018

I have tested this item 🔴 unsuccessfully on 5abf679

~~Got at administrator/index.php?option=com_config Component not found. ~~

Also can't uninstall Patch PatchTester\Model\PullModel::revert - Error retrieving table data ([])


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/21341.
avatar dgrammatiko
dgrammatiko - comment - 1 Aug 2018

@franz-wohlkoenig you cannot test this with patchtester, you need to fetch/pull my branch

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 1 Aug 2018 - Not tested
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 1 Aug 2018

I have not tested this item.


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

avatar wilsonge wilsonge - change - 6 Aug 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-08-06 09:35:40
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 6 Aug 2018
avatar wilsonge wilsonge - merge - 6 Aug 2018

Add a Comment

Login with GitHub to post a comment