Skip to content

Feature/[SD-1482][SD-1483] MFA setup enhancements #730

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
MdNadimHossain wants to merge 17 commits into
base: develop
Choose a base branch
from
Open

Feature/[SD-1482][SD-1483] MFA setup enhancements #730

MdNadimHossain wants to merge 17 commits into from

Conversation

@MdNadimHossain
Copy link
Contributor

@MdNadimHossain MdNadimHossain commented Jan 29, 2026 •
edited
Loading

JIRA issue:
https://digital-vic.atlassian.net/browse/SD-1482
https://digital-vic.atlassian.net/browse/SD-1483

Changed

  1. Added custom work and override function to change the logic of tfa email otp to send the email when the verification form loads for the first time
  2. Changes label and text to reflect multi-factor authentication for non admin access pages
  3. Modified the multi-factor authentication related status messages
  4. Added new view password module, this add the eye icon plugin for the tfa setup page where the user needs to enter the password
  5. Added update hook to roll out the necessary updates to all projects
  6. Added behat test

Screenshots

@MdNadimHossain MdNadimHossain self-assigned this Jan 29, 2026
@MdNadimHossain
[SD-1482] Added changes for tfa setup form and enabaled view password…
ba29307 
… module.
@MdNadimHossain
[SD-1482] Added Email setup from changes.
8c5cfdc
@MdNadimHossain
[SD-1482] Added breadcrumb changes and some label, title and descript…
4b02a1e 
…ion updates.
@MdNadimHossain
[SD-1482] Updated dashboard module to redirect to tfa setup form afte…
fdda4c9 
…r user login and lint fix.
@MdNadimHossain
[SD-1482] Fixed type miss match error during build and lint fix.
bcda8e0
@MdNadimHossain
[SD-1482] Fixed type miss match error during build for view password …
4eb2a59 
…settings.
@MdNadimHossain
[SD-1482] Added custom messages and replaced tfa module messages.
0388028
@MdNadimHossain
Merge branch 'develop' into feature/SD-1482-mfa-setup-enhancements
96916a5
@MdNadimHossain
lint fix.
bc30872
@MdNadimHossain
[SD-1483] Added Multifactor authentication verification enhancements.
1831efc
@MdNadimHossain
[SD-1483] lint fix.
f8ca4a4
@MdNadimHossain
Added hook to update email templates for TFA and TFA email otp.
dd66227
@MdNadimHossain MdNadimHossain force-pushed the feature/SD-1482-mfa-setup-enhancements branch from 64300bf to dd66227 Compare February 11, 2026 01:42
@MdNadimHossain MdNadimHossain changed the title Feature/sd 1482 mfa setup enhancements Feature/[SD-1482][SD-1483] MFA setup enhancements Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vincent-gao vincent-gao Awaiting requested review from vincent-gao

@lambry lambry Awaiting requested review from lambry

@anthony-malkoun anthony-malkoun Awaiting requested review from anthony-malkoun

At least 1 approving review is required to merge this pull request.

Assignees

@MdNadimHossain MdNadimHossain

Labels

Needs review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MdNadimHossain