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.

Sign up for GitHub

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

Jump to bottom

WIC/ECE Treatment 1 - links #458

Merged
merged 5 commits into from
Dec 18, 2023
Merged

WIC/ECE Treatment 1 - links #458

merged 5 commits into from
Dec 18, 2023

Conversation

sree-cfa
Copy link
Contributor

@sree-cfa sree-cfa commented Dec 18, 2023 •
edited
Loading

Adding screens for treatment group 1 - "links" and conditional navigation for experiment
Figma designs

Does not include Treatment group 2 - apply. This will be added in a follow-up PR

Testing Steps

  1. Select parish (NOLA vs not-NOLA)
  2. Trigger treatment grouping - this is currently random, might need to trigger it a few times to get the LINKS treatment
  3. Jump to screen before wic/ece screens
  4. Verify WIC/ECE screens - depends on treatment
  5. Jump to signature page and continue to confirmation
  6. Verify confirmation blocks - depends on treatment

@sree-cfa sree-cfa temporarily deployed to test-la-doc-uploader-pr-458 December 18, 2023 19:55 Inactive
@sree-cfa sree-cfa temporarily deployed to test-la-doc-uploader-pr-458 December 18, 2023 19:59 Inactive
@sree-cfa sree-cfa temporarily deployed to test-la-doc-uploader-pr-458 December 18, 2023 20:18 Inactive
@sree-cfa sree-cfa marked this pull request as ready for review December 18, 2023 20:22
@sree-cfa sree-cfa temporarily deployed to test-la-doc-uploader-pr-458 December 18, 2023 20:22 Inactive
@sree-cfa sree-cfa added the ready for review This PR is ready for code-review label Dec 18, 2023
@analoo analoo self-requested a review December 18, 2023 23:23
analoo
analoo approved these changes Dec 18, 2023
View reviewed changes
Copy link
Contributor

@analoo analoo left a comment

Choose a reason for hiding this comment

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

LGTM

@sree-cfa sree-cfa merged commit 7587ab6 into main Dec 18, 2023
5 checks passed
@sree-cfa sree-cfa deleted the treatment-links branch December 18, 2023 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@analoo analoo analoo approved these changes

Assignees
No one assigned
Labels
ready for review This PR is ready for code-review
Projects
None yet
Milestone
No milestone
2 participants
@sree-cfa @analoo