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

Remove UTM Type Column #6

Merged
merged 1 commit into from
Mar 21, 2023
Merged

Remove UTM Type Column #6

merged 1 commit into from
Mar 21, 2023

Conversation

jmhollinger
Copy link

Description & motivation

UTM Type is a non-standard UTM column.Removing the static references.

Additional columns can be passed through using the project variable segment_pass_through_columns.

Fixes #5

Checklist

  • I have verified that these changes work locally
  • I have updated the README.md (if applicable)
  • I have added tests & descriptions to my models (and macros if applicable)

UTM Type is a non-standard UTM column. Removing the static reference. Additional columns can be passed through using the project variable `segment_pass_through_columns`. Fixes #5
@jmhollinger jmhollinger added the bug Something isn't working label Mar 21, 2023
@jmhollinger jmhollinger requested a review from jmmizerany March 21, 2023 14:21
@jmhollinger jmhollinger merged commit c27b167 into main Mar 21, 2023
@jmhollinger jmhollinger deleted the remove-utm-type branch March 21, 2023 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CONTEXT_CAMPAIGN_TYPE is not a field on Segment's page table
2 participants