Skip to content

Add A/B test buckets to idv_phone_of_record_visited#8864

Merged
soniaconnolly merged 1 commit intomainfrom
sonia-lg-10345-add-ab-test-buckets-to-more-analytics
Jul 25, 2023
Merged

Add A/B test buckets to idv_phone_of_record_visited#8864
soniaconnolly merged 1 commit intomainfrom
sonia-lg-10345-add-ab-test-buckets-to-more-analytics

Conversation

@soniaconnolly
Copy link
Contributor

@soniaconnolly soniaconnolly commented Jul 25, 2023

🎫 Ticket

LG-10345

🛠 Summary of changes

We want to divide the funnel events by whether the user saw Welcome or GettingStarted for the GettingStarted A/B test, so add the bucket to analytics event IdV: phone of record visited.

📜 Testing Plan

  • In application.yml add idv_getting_started_a_b_testing: '{"welcome":0, "getting_started":100 }'
  • Create account, start IdV
  • Continue past phone step
  • Check events.log that IdV: phone of record visited has attributes acuant_sdk_upgrade_ab_test_bucket: :default, getting_started_ab_test_bucket: :getting_started
  • Check dashboard sc-idv-getting-started-ab-test with the dev events log once this is merged.

We want to divide the funnel events by whether the user saw Welcome or GettingStarted

[skip changelog]
@soniaconnolly soniaconnolly requested a review from a team July 25, 2023 22:05
Copy link
Contributor

@matthinz matthinz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@soniaconnolly soniaconnolly merged commit ba0bdd0 into main Jul 25, 2023
@soniaconnolly soniaconnolly deleted the sonia-lg-10345-add-ab-test-buckets-to-more-analytics branch July 25, 2023 22:48
mitchellhenke pushed a commit that referenced this pull request Jul 26, 2023
We want to divide the IdV funnel events by whether the user saw Welcome or GettingStarted, so add A/B test buckets to idv_phone_of_record_visited.

[skip changelog]
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.

2 participants