-
Notifications
You must be signed in to change notification settings - Fork 40
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
Add create_or_update_subscribers method #15
Add create_or_update_subscribers method #15
Conversation
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, thanks!
Thanks for the submission! I'm going to see if I can get another pair of eyes on this internally, then assuming there are no issues, I'll merge and push it out. |
src/Client.php
Outdated
* @param array $params | ||
* @return \Drip\ResponseInterface | ||
*/ | ||
public function create_or_update_subscriber_batch($params) |
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.
Looking at this again, could we change the name of this method to create_or_update_subscribers
to match the Ruby client? https://github.com/DripEmail/drip-ruby/blob/master/lib/drip/client/subscribers.rb#L67
Thanks!
7e12b91
to
858dd09
Compare
I renamed the method, thanks for the quick feedback. |
This just got released in 1.1.0. |
Uses the Drip REST API batch endpoint to update up to 1000 subscribers at a time.
http://developer.drip.com/#batch-api