-
Notifications
You must be signed in to change notification settings - Fork 2
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
[LD-1386] Fix Python scripts and add reveal_phone_number + webhook-url fields #72
Conversation
@@ -35,23 +37,25 @@ url = "https://api.apollo.io/api/v1/people/bulk_match" | |||
|
|||
data = { | |||
"api_key": "YOUR API KEY HERE", | |||
"reveal_personal_emails": true, | |||
"reveal_personal_emails": True, |
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.
Please keep this as true instead of True. Same way for other places as well
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.
This is actually wrong. Python boolean representation is True
and not true
Therefore, when a request is made from python, we should have True.
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.
Please change the boolean value
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.
LGTM
generated from commit 334e844
Make sure you've checked off all these things before submitting:
https://apollopde.atlassian.net/browse/LD-1386
Core Incident: https://apollopde.atlassian.net/browse/INCIDENT-8522
reveal_phone_number
andwebhook_url
fieldBefore:
After
master
branch.