?
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