Skip to content

Commit

Permalink
changelog: Internal, Face or Touch Unlock, Logging for users in FT un…
Browse files Browse the repository at this point in the history
…lock setup ab test
  • Loading branch information
kevinsmaster5 committed Dec 20, 2024
1 parent 75f02ad commit 3f26025
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ def index
@presenter = two_factor_options_presenter
analytics.user_registration_2fa_setup_visit(
enabled_mfa_methods_count:,
in_account_creation_flow: user_session[:in_account_creation_flow] || false,
gov_or_mil_email: fed_or_mil_email?,
)
end
Expand Down
3 changes: 3 additions & 0 deletions app/services/analytics_events.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7326,15 +7326,18 @@ def user_registration_2fa_setup(
# Tracks when user visits MFA selection page
# @param [Integer] enabled_mfa_methods_count Number of enabled MFA methods on the account
# @param [Boolean] gov_or_mil_email Whether registered user has government email
# @param [Boolean] in_account_creation_flow Whether user is going through account creation flow
def user_registration_2fa_setup_visit(
enabled_mfa_methods_count:,
gov_or_mil_email:,
in_account_creation_flow: nil,
**extra
)
track_event(
'User Registration: 2FA Setup visited',
enabled_mfa_methods_count:,
gov_or_mil_email:,
in_account_creation_flow:,
**extra,
)
end
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
'User Registration: 2FA Setup visited',
enabled_mfa_methods_count: 0,
gov_or_mil_email: false,
in_account_creation_flow: false,
)
end

Expand Down Expand Up @@ -47,6 +48,7 @@
'User Registration: 2FA Setup visited',
enabled_mfa_methods_count: 0,
gov_or_mil_email: true,
in_account_creation_flow: false,
)
end
end
Expand Down Expand Up @@ -86,6 +88,7 @@
'User Registration: 2FA Setup visited',
enabled_mfa_methods_count: 1,
gov_or_mil_email: false,
in_account_creation_flow: false,
)
end
end
Expand Down

0 comments on commit 3f26025

Please sign in to comment.