Skip to content

Conversation

@grandmaester
Copy link
Contributor

@grandmaester grandmaester commented Nov 28, 2025

Description

This PR integrates mfa-push-enrollment-qr screen with continue method which gets invoked once user completes qr code registration via Auth0 Guardian or similar authenticator application.

Changes

Changes made on below files:
1: MfaPushEnrollmentQRForm(react/src/screens/mfa-push-enrollment-qr/components/MfaPushEnrollmentQRForm.tsx) for incorporating handle continue hook call
2. useMfaPushEnrollmentQRManager(react/src/screens/mfa-push-enrollment-qr/hooks/useMfaPushEnrollmentQRManager.ts) for adding a new wrapper over continue sdk method
3. createMockMfaPushEnrollmentQRInstance(react/src/mocks/@auth0/auth0-acul-react/mfa-push-enrollment-qr.ts) for adding mocks for relevant changes

  • List the specific changes made
  • Include any breaking changes
  • Note any new dependencies or configuration updates

Testing

Screen Tests:
image

image image

Unit Tests:
image

image

Build Screenshot:
image

Lint Test:
image

Manifest Validate:
image

  • Tests pass locally
  • New tests added for new functionality
  • Manual testing completed
  • No breaking changes to existing functionality

Checklist

  • Code follows the project's style guidelines
  • Self-review of the code completed
  • Code is commented where necessary
  • Documentation updated if needed
  • Changes generate no new warnings

@grandmaester grandmaester self-assigned this Nov 28, 2025
@grandmaester grandmaester changed the title fix: mfa enrollment qr integrate with continue method fix: MFA Enrollment QR Screen Integration with continue method of SDK Nov 28, 2025
@grandmaester grandmaester merged commit 7490aa2 into main Nov 28, 2025
3 checks passed
@grandmaester grandmaester deleted the fix/mfa-push-enrollment-qr-updates-with-sdk branch November 28, 2025 12:07
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.

3 participants