Skip to content

Implemented Multi-factor Authentication for 1110-vue3 #509

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 8 commits into
base: 1110-vue3
Choose a base branch
from
Open

Implemented Multi-factor Authentication for 1110-vue3 #509

wants to merge 8 commits into from

Conversation

@tiwari-nishant
Copy link

@tiwari-nishant tiwari-nishant commented Nov 4, 2025

@tiwari-nishant tiwari-nishant self-assigned this Nov 4, 2025
Nikhil-Ashoka
Nikhil-Ashoka reviewed Nov 7, 2025
View reviewed changes
@Nikhil-Ashoka
Copy link
Collaborator

Nikhil-Ashoka commented Nov 7, 2025

Different Error message.
Spaces is not uniform between each input boxes
Ok is displayed instead of Login
Click on Cancel in Register Modal and then Click login in Login page, TOTP is already showing as error.
Secret key toggle button is not properly aligned.
MFA bypass not working in Modal User. User is created with MFA bypass disabled always.
Login page, Cannot submit OTP
UI is breaking after clicking on Edit user and then clicking on Add user.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Nikhil-Ashoka Nikhil-Ashoka Nikhil-Ashoka left review comments

@vedangimittal vedangimittal Awaiting requested review from vedangimittal

Assignees

@tiwari-nishant tiwari-nishant

Labels

1110-vue3 vue3

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tiwari-nishant @Nikhil-Ashoka