Skip to content

LG-8274: In-Person Proofing CTA A/B Testing#7669

Merged
jess-fortier merged 45 commits intomainfrom
jess/LG-8274-IPP-CTA-AB-test-higher-level-switching
Jan 27, 2023
Merged

LG-8274: In-Person Proofing CTA A/B Testing#7669
jess-fortier merged 45 commits intomainfrom
jess/LG-8274-IPP-CTA-AB-test-higher-level-switching

Conversation

@sheldon-b
Copy link
Contributor

@sheldon-b sheldon-b commented Jan 20, 2023

🎫 Ticket

LG-8274: Try in Person IPP CTA A/B Testing

🛠 Summary of changes

  • Introduce structure for Login IPP variant testing, which doubles to support future throttling work
  • Implement variant A view with nationwide language with translations
  • Implement variant B view with nationwide language with translations
  • Implement throttling view (variant C) that hides IPP entrypoint when IPP is enabled
  • Ensure existing RIDP flow is unchanged when IPP or AB testing are disabled
  • Log active variant for each user session

📜 Testing Plan

  1. Step through flow and visually confirm expected variant view and copy under all combinations of
  • in_person_proofing_enabled true / false
  • in_person_cta_variant_testing_enabled true / false
  • in_person_cta_variant_testing_percents values:
    -- {}
    -- {"A":100, "B":0}
    -- {"A":0,"B":100}
    -- {"A":0,"B":0,"C":100}
    -- Document authentication error type (hard fail, or attention)
    -- Language
  1. With IPP and variant testing enabled and testing percents as {"A":45,"B":45,"C":10}, step through the doc auth attention error flow with multiple user sessions and confirm that variant will change.

  2. Ensure existing automated test suite passes

👀 Screenshots

Seen for all test scenarios in google drive here: https://docs.google.com/document/d/1WxmfnSmoaKVblFd2eio9OA71ONaIRJITdw1dn6jxwEg/edit?usp=sharing

jess-fortier and others added 25 commits January 12, 2023 15:08
…e the variant B Try in person button to the IPP location page. Remove unnecessary comments for IPP variant testing. Log an event when each variant is shown.
…arch About text to reflect additional POs coming available soon and CTA Prompt Detail to include comma before D.C. (all languages)
…s for improved in-person proofing calls to action
@jess-fortier jess-fortier changed the title Jess/lg 8274 ipp cta ab test higher level switching Jess/lg 8274 - IPP CTA A/B Testing Jan 23, 2023
@sheldon-b sheldon-b changed the title Jess/lg 8274 - IPP CTA A/B Testing LG-8274: In-Person Proofing CTA A/B Testing Jan 23, 2023
heading,
actionText,
actionOnClick,
altActionText,
Copy link
Contributor

Choose a reason for hiding this comment

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

just want to make sure i understand - altAction is variant-b's cta?

Copy link
Contributor

Choose a reason for hiding this comment

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

Speaking first generally, we augmented this component to allow for two actions in the same paradigm as you might see an "OK" and a "Cancel". The primary user action is as before: Blue button, white text. We've added an alternative user action option that is presented as a white button with blue outline.

You are correct about the way this is used in the variant B Warning component.

Copy link
Contributor

Choose a reason for hiding this comment

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

ahh ok so then i should think of this as a [stylistic] secondary version of the button.

Copy link
Contributor

Choose a reason for hiding this comment

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

Hm I might be misunderstanding you here - "a [stylistic] secondary version of the button" makes it sound like a different (stylistic) way to present a sole intended user option, like you would be using the white-with-blue-outline style instead of the blue. While the implementation would technically allow for that (I think...), that isn't the intended use. Rather than a different way to display the same button, we wanted the Warning to display two different buttons.

/**
* The specific A/B testing variant that was activated for the current user session.
*/
inPersonCtaVariantActive?: string;
Copy link
Contributor

Choose a reason for hiding this comment

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

what's the difference btwn enabled and active? Is enabled saying "a/b testing happened" and active is saying "this user saw variant x"? If so can we change the naming to something like inPersonCtaVariantVersion? active makes me think it would be a boolean value.

Copy link
Contributor

Choose a reason for hiding this comment

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

oh, or just inPersonCtaVariant?

Copy link
Contributor

Choose a reason for hiding this comment

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

Valid question - I went back and forth on the naming of these for a long time. Let me know if you have a suggestion for how to better capture what's happening here:
inPersonCtaVariantTestingEnabled - Is AB testing turned on in the application config?
inPersonCtaVariantActive - Which variant has been activated for the current user session?

Copy link
Contributor

@svalexander svalexander Jan 26, 2023

Choose a reason for hiding this comment

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

i think the enabled one is clear. The active one i would change to something like inPersonCtaVariant or even just switch around variant and active to inPersonCtaActiveVariant. I know when i see an action word at the end of a variable name it suggests a boolean to me.

Copy link
Contributor

@tomas-nava tomas-nava left a comment

Choose a reason for hiding this comment

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

approve!

@jess-fortier jess-fortier merged commit 3080b01 into main Jan 27, 2023
@jess-fortier jess-fortier deleted the jess/LG-8274-IPP-CTA-AB-test-higher-level-switching branch January 27, 2023 15:38
@zachmargolis zachmargolis mentioned this pull request Jan 30, 2023
@aduth aduth mentioned this pull request Jun 8, 2023
4 tasks
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.

6 participants