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

Adding a job that will send users with canonical_donor_id to BigQuery #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

accessmayur1
Copy link

Summary:
Created a function that can call Sailthru's /job api with job=export_list_data.
This will allow us to get any user list data from sailthru and push it into BigQuery

The reason for this change is to allow us to associate users that have a canonical_donor_id with a profile_id, which is generated by sailthru
Ultimately this improves our donation attribution for the EOY Campaign

Issue: https://khanacademy.atlassian.net/browse/MP-2102

Test Plan:

  • Tested dry-run locally with a small set of "test" data
  • Tested pushing data to BigQuery from local machine
  • Tested real list data dry-run
  • Tested real list data pushed to BigQuery from local machine

Reviewers: russell, catyanish, ragini, csilvers

Reviewed By: ragini

Differential Revision: https://phabricator.khanacademy.org/D58815

Summary:
Created a function that can call Sailthru's /job api with job=export_list_data.
This will allow us to get any user list data from sailthru and push it into BigQuery

The reason for this change is to allow us to associate users that have a canonical_donor_id with a profile_id, which is generated by sailthru.
Ultimately this improves our donation attribution for the EOY Campaign

Issue: https://khanacademy.atlassian.net/browse/MP-2102

Test Plan:
- Tested dry-run locally with a small set of "test" data
- Tested pushing data to BigQuery from local machine
- Tested real list data dry-run
- Tested real list data pushed to BigQuery from local machine

Reviewers: russell, catyanish, ragini, csilvers

Reviewed By: ragini

Differential Revision: https://phabricator.khanacademy.org/D58815
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.

1 participant