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

feat: new acquisition query for attributing organic, paid #1108

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

langswei
Copy link
Collaborator

Categorize page views by acquisition type (organic, paid) and traffic source (search, social, email, etc).

Related Slack thread at https://cq-dev.slack.com/archives/C04MQCRHLBY/p1693113460204539.

Copy link

@ramboz ramboz left a comment

Choose a reason for hiding this comment

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

Looks good to me!

I'd probably just remove the utm_term part as this will be removed from the RUM collection

src/queries/rum-acquisition.sql Outdated Show resolved Hide resolved
Copy link

This PR will trigger a minor release when merged.

@langswei langswei requested a review from dzehnder April 24, 2024 11:36
@ramboz
Copy link

ramboz commented May 14, 2024

@trieloff Seeing the documentation was added to the PR. Is there anything else blocking this from your point of view?

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.

None yet

3 participants