Skip to content

[FIX] User is still asked for 2FA confirmation even if it is deactivated#22801

Merged
pierre-lehnen-rc merged 2 commits intodevelopfrom
fix/disabled-2fa
Aug 6, 2021
Merged

[FIX] User is still asked for 2FA confirmation even if it is deactivated#22801
pierre-lehnen-rc merged 2 commits intodevelopfrom
fix/disabled-2fa

Conversation

@pierre-lehnen-rc
Copy link
Copy Markdown
Contributor

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

Copy link
Copy Markdown
Contributor

@ostjen ostjen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@pierre-lehnen-rc pierre-lehnen-rc merged commit 89ee544 into develop Aug 6, 2021
@pierre-lehnen-rc pierre-lehnen-rc deleted the fix/disabled-2fa branch August 6, 2021 18:54
gabriellsh added a commit that referenced this pull request Aug 10, 2021
* 'develop' of github.com:RocketChat/Rocket.Chat:
  [FIX] Manually approve new users is not applied to SAML users (#22823)
  [FIX] applyChatRestictions callback not working for community version (#22839)
  [FIX] Message update not working in some cases (#22856)
  [FIX] Broken download link on uploaded files (#22848)
  [FIX] User is still asked for 2FA confirmation even if 2FA is globally deactivated. (#22801)
  [FIX] Registration not possible with TOTP and email verification (#22778)
  [FIX] users registered via third party apps bypass custom required fields (#22396)
  [FIX] "Click to Join" button is not working if there are no muted users in the room (#22871)
  [BREAK][ENTERPRISE] CSV file downloaded in the Engagement Dashboard's New Users section contains undefined data (#22862)
  Fix Local time conversion in the Users By Time of the Day chart (#22836)
@sampaiodiego sampaiodiego mentioned this pull request Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants