?
avatar PhilETaylor
PhilETaylor
8 Jun 2020

Steps to reproduce the issue

Enable the 2FA plugins in Joomla 4
visit the users list

Expected result

Consistent use of icons ticks and crosses

Actual result

different X and tick for non enabled two factor users

Screenshot 2020-06-08 at 19 34 13

avatar PhilETaylor PhilETaylor - open - 8 Jun 2020
avatar joomla-cms-bot joomla-cms-bot - change - 8 Jun 2020
Labels Added: ?
avatar PhilETaylor PhilETaylor - change - 8 Jun 2020
Status New Closed
Closed_Date 0000-00-00 00:00:00 2020-06-08 18:34:50
Closed_By PhilETaylor
avatar joomla-cms-bot joomla-cms-bot - labeled - 8 Jun 2020
avatar PhilETaylor PhilETaylor - close - 8 Jun 2020
avatar PhilETaylor PhilETaylor - change - 8 Jun 2020
Status Closed New
Closed_Date 2020-06-08 18:34:50
Closed_By PhilETaylor
avatar PhilETaylor PhilETaylor - reopen - 8 Jun 2020
avatar N6REJ
N6REJ - comment - 9 Jun 2020

pretty sure this is fixed by #28075

avatar PhilETaylor
PhilETaylor - comment - 9 Jun 2020

pretty sure this is fixed by #28075

Nope...

Screenshot 2020-06-09 at 19 19 43

Screenshot 2020-06-09 at 19 20 30

avatar Quy
Quy - comment - 9 Jun 2020

Fixed in #28075 pending to be merged.

29508

avatar PhilETaylor
PhilETaylor - comment - 9 Jun 2020

well I applied that patch with the patch tester and it did not resolve the problem for me. I'll try it again

avatar Quy
Quy - comment - 9 Jun 2020

Sometimes Patch Tester fails. You can use the download package/installer in the footer of the PR. Click Show all checks link where there is a Download link. https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/28075/downloads/32886/

avatar PhilETaylor PhilETaylor - change - 9 Jun 2020
Status New Closed
Closed_Date 0000-00-00 00:00:00 2020-06-09 20:23:57
Closed_By PhilETaylor
avatar PhilETaylor
PhilETaylor - comment - 9 Jun 2020

Well if you are sure, I take your word for it.

avatar PhilETaylor PhilETaylor - close - 9 Jun 2020
avatar Quy
Quy - comment - 9 Jun 2020

You can see in my screenshot. I will keep an eye on this issue until if/when #28075 is merged. Thanks for the report.

avatar N6REJ
N6REJ - comment - 10 Jun 2020

@PhilETaylor patch tester doesn't do npm ci which is needed in this case.

avatar PhilETaylor
PhilETaylor - comment - 10 Jun 2020

Thank you

Sent from my iPhone

On 10 Jun 2020, at 01:03, Bear notifications@github.com wrote:


@PhilETaylor patch tester doesn't do npm ci which is needed in this case.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Add a Comment

Login with GitHub to post a comment