? ? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
24 Sep 2020

Summary of Changes

Backport 2FA edit screen for com_admin from 4.x to 3.x too

Testing Instructions

Actual result BEFORE applying this Pull Request

It is not possible to configure 2fa from com_admin

Expected result AFTER applying this Pull Request

We can configure 2fa from com_admin

Documentation Changes Required

None

avatar zero-24 zero-24 - open - 24 Sep 2020
avatar zero-24 zero-24 - change - 24 Sep 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Sep 2020
Category Administration com_admin
avatar zero-24 zero-24 - change - 24 Sep 2020
The description was changed
avatar zero-24 zero-24 - edited - 24 Sep 2020
avatar gogicomputers gogicomputers - test_item - 24 Sep 2020 - Tested successfully
avatar gogicomputers
gogicomputers - comment - 24 Sep 2020

I have tested this item successfully on 2522d70

Testet with Joomla 3.9.21. Patch works.


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

avatar viocassel viocassel - test_item - 24 Sep 2020 - Tested successfully
avatar viocassel
viocassel - comment - 24 Sep 2020

I have tested this item successfully on 2522d70


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

avatar richard67
richard67 - comment - 24 Sep 2020

@zero-24 Please fix PHP code style errors reported by drone here https://ci.joomla.org/joomla/joomla-cms/35668/1/3 .

avatar richard67
richard67 - comment - 24 Sep 2020

PR should not be RTC before PHP code style errors are fixed.

avatar zero-24 zero-24 - change - 24 Sep 2020
Labels Added: ?
avatar zero-24
zero-24 - comment - 24 Sep 2020

Should be fixed now thanks @richard67

avatar richard67
richard67 - comment - 24 Sep 2020

@zero-24 Normally you can check drone results yourself. Let me know if you need some information on that.

avatar richard67 richard67 - alter_testresult - 24 Sep 2020 - gogicomputers: Tested successfully
avatar richard67 richard67 - alter_testresult - 24 Sep 2020 - viocassel: Tested successfully
avatar zero-24
zero-24 - comment - 24 Sep 2020

I have fixed them just a minute ago right?

avatar richard67 richard67 - change - 24 Sep 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 24 Sep 2020

Previous test results were still valid because change after that was only code style of comments => RTC.


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

avatar richard67
richard67 - comment - 24 Sep 2020

@zero-24 I just meant you could check yourself if there are errors after a commit. Just a hint for improvement, nothing bad.

avatar richard67 richard67 - change - 24 Sep 2020
Status Ready to Commit Pending
avatar richard67
richard67 - comment - 24 Sep 2020

New PHPCS error => back to pennding.


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

avatar zero-24
zero-24 - comment - 24 Sep 2020

Patched thanks :)

avatar richard67 richard67 - alter_testresult - 24 Sep 2020 - gogicomputers: Tested successfully
avatar richard67 richard67 - alter_testresult - 24 Sep 2020 - viocassel: Tested successfully
avatar richard67 richard67 - change - 24 Sep 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 24 Sep 2020

RTC


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

avatar richard67
richard67 - comment - 24 Sep 2020

@zero-24 Sorry if I sounded harsh or so. Was not meant like this. I have to learn to be a bit more patient.

avatar zero-24
zero-24 - comment - 24 Sep 2020

All fine :-)

avatar rdeutz rdeutz - change - 26 Sep 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-09-26 14:44:10
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 26 Sep 2020
avatar rdeutz rdeutz - merge - 26 Sep 2020

Add a Comment

Login with GitHub to post a comment