Skip to content

LG-14887: Log action name with reCAPTCHA assessment result received#11407

Merged
aduth merged 2 commits intomainfrom
aduth-lg-14887-recaptcha-action-logging
Oct 28, 2024
Merged

LG-14887: Log action name with reCAPTCHA assessment result received#11407
aduth merged 2 commits intomainfrom
aduth-lg-14887-recaptcha-action-logging

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Oct 28, 2024

🎫 Ticket

LG-14887

🛠 Summary of changes

Updates reCAPTCHA analytics logging to include the expected "action" name, so that we can more clearly differentiate reCAPTCHA results between sign-in and phone setup.

📜 Testing Plan

  1. In a separate terminal process, run make watch_events
  2. Go to http://localhost:3000 in a private browsing window
  3. Sign in to an account
  4. In the make watch_events output, find the event for "reCAPTCHA verify result received", and observe the expected properties.event_properties.recaptcha_action: 'sign_in'
  5. Optionally, repeat these same steps for the phone setup flow for international numbers, and verify the expected value of phone_setup

changelog: Internal, Analytics, Log action name with reCAPTCHA assessment result received`
@aduth aduth requested a review from a team October 28, 2024 13:21
@aduth aduth changed the title LG-14887: Log action name with reCAPTCHA assessment result received` LG-14887: Log action name with reCAPTCHA assessment result received Oct 28, 2024
@aduth aduth merged commit 1e55a4b into main Oct 28, 2024
@aduth aduth deleted the aduth-lg-14887-recaptcha-action-logging branch October 28, 2024 15:45
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