keegan
Novembro 8, 2024, 12:20am
2
That’s a reasonable request, I agree, it would be good to have a confirmation dialog for this action here. Sorry for the inconvenience this might’ve caused. I’ve just made a PR fixing the issue:
discourse:main ← discourse:disable-2fa-confirm
opened 12:19AM - 08 Nov 24 UTC
## :mag: Overview
This PR ensures that admins are shown a confirmation dialog w… hen clicking to disable 2FA for a user. The 2FA button is right below the "Grant Badge" button and as such it can easily be clicked accidentally. It's also good practice to ask for confirmation before removing important functionality.
## 🔗 More details
As requested on Meta:
https://meta.discourse.org/t/confirmation-dialog-on-admin-2fa-disable-button/335039
## 📷 Screenshots
<img width="608" alt="Screenshot 2024-11-07 at 13 24 21" src="https://github.com/user-attachments/assets/32b74f4b-3fa1-4010-8bd6-6260abb703f4">
3 curtidas