-
Notifications
You must be signed in to change notification settings - Fork 117
feat: build win10 user targeting for braze to start ip warming #8135
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
Conversation
I need to be able to to query |
I will open a ticket to get myself access to |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
sql/moz-fx-data-shared-prod/braze_derived/fxa_win10_users_v1/metadata.yaml
Outdated
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
OKAY! I have made some changes. Ultimately while I don't think we need an intermediate table, I decided that I would like to have one to separate the steps of what is happening:
The daily and historical tables could hypothetically be a single table, but I like the separation of their functions. I think it also allows us more flexibility in terms of what we will need to change if approval is granted to send follow up emails. |
The above tables can all be view in BQ. I ran a few days worth of backfill and made sure that we did not have duplicate in the historical table. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great work, @LiamMcFall 🙌
This comment has been minimized.
This comment has been minimized.
Integration report for "Merge branch 'main' into win10_users_activation"
|
Description
This PR creates the basis of what we need to target 14 day inactive windows 10 firefox users.
Related Tickets & Documents
Reviewer, please follow this checklist