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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃獰聽馃敡 Remove utmStorage util #22194

Merged
merged 1 commit into from
Feb 1, 2023

Conversation

letiescanciano
Copy link
Contributor

This method was only effective when we redirect users directly to cloud.airbyte.io/signup?utm_source..... but wasn't working when users first landed in airbyte.com with UTM params and then end up signing up.

We are managing UTM params with cross domain cookies in the different tools we use to measure attribution (GA and Hubspot) so we don't need this anymore.

This method was only effective when we redirect users directly to
cloud.airbyte.io/signup?utm_source..... but wasn't working when users
first landed in airbyte.com with UTM params and then end up signing up.

We are managing UTM params with cross domain cookies in the different
tools we use to measure attribution (GA and Hubspot) so we don't need
this anymore.
@octavia-squidington-iv octavia-squidington-iv added the area/frontend Related to the Airbyte webapp label Feb 1, 2023
Copy link
Collaborator

@timroes timroes 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 not tested locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Related to the Airbyte webapp team/growth
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants