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.

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

Send email-required flag to signup endpoint #140

Merged
merged 1 commit into from
Feb 29, 2024
Merged

Conversation

oyamauchi
Copy link
Member

Description

We'll want to track this along with signups to measure the effect of
requiring email. This will require a followup to Zuplo to read this
param and send it to SFMC.

Test Plan

Sign up under both conditions, make sure the flag is present and with
the correct value in network inspector. (Zuplo isn't reading the flag
yet so nothing will change in SFMC.)

## Description

We'll want to track this along with signups to measure the effect of
requiring email. This will require a followup to Zuplo to read this
param and send it to SFMC.

## Test Plan

Sign up under both conditions, make sure the flag is present and with
the correct value in network inspector. (Zuplo isn't reading the flag
yet so nothing will change in SFMC.)
Copy link

vercel bot commented Feb 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
embed-rewiringamerica-org ✅ Ready (Inspect) Visit Preview Feb 28, 2024 9:20pm

Copy link

@kylehotchkiss kylehotchkiss left a comment

Choose a reason for hiding this comment

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

Code LGTM. Have you been able to successfully submit and verify the value shows in SFMC?

@oyamauchi
Copy link
Member Author

Not yet, but I have verified the value successfully gets to Zuplo, so I'm going to go ahead and land this.

@oyamauchi oyamauchi merged commit 2c75dbd into main Feb 29, 2024
3 checks passed
@oyamauchi oyamauchi deleted the owen.required branch February 29, 2024 03:33
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