Skip to content
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.

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

LG-10965: Show backup code reminder for partner-initiated requests #11744

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aduth
Copy link
Member

@aduth aduth commented Jan 14, 2025

🎫 Ticket

LG-10965

🛠 Summary of changes

Updates backup code reminder screen to use new controller actions implemented in #11738 to support use with sign-ins associated with a partner application, and updated after-sign-in logic to include partner-associated sign-ins to be considered for backup code reminder.

Temporarily merges to #11738. This cannot be merged until those changes are fully deployed to production.

📜 Testing Plan

Repeat testing plan from #11738, but also verify that it shows reminder when initiating sign-in from partner application (e.g. sample OIDC application), and that you eventually return to the partner application after making your reminder choice.

👀 Screenshots

Note the partner logo in the screenshot, indicating that it is a partner-associated sign-in request:

Screenshot 2025-01-14 at 8 34 16 AM

@aduth aduth requested a review from a team January 14, 2025 14:04
@aduth aduth force-pushed the aduth-lg-10965-backup-code-reminder branch from 87c58ac to ff75b13 Compare January 14, 2025 16:45
Base automatically changed from aduth-lg-10965-backup-code-reminder to main January 14, 2025 20:55
@aduth aduth force-pushed the aduth-lg-10965-backup-code-reminder-auth branch from 8cdbe61 to c1e2c48 Compare January 14, 2025 20:56
changelog: User-Facing Improvements, Backup Codes, Show backup code reminder for partner-initiated requests
@aduth aduth force-pushed the aduth-lg-10965-backup-code-reminder-auth branch from c1e2c48 to 99242b7 Compare January 15, 2025 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant