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

Twitter ads domain fix #593

Closed
wants to merge 2 commits into from
Closed

Twitter ads domain fix #593

wants to merge 2 commits into from

Conversation

willemverspyck
Copy link

Add support for Twitter Ads API:

  • Has different API host: https://ads-api.twitter.com. Possible to change by setApiHost($host)
  • Has different API version: 2 (or 1 for older versions of Twitter Ads API). Possible to change by setApiVersion($version)
  • Has different extension: No default suffix ".json". Possible to change by setApiExtension($extension)
  • Throws error when httpCode is not equal to 200 on request. For example httpCode 429 is when rate limit has reached.

@abraham abraham removed the Explore label Jul 2, 2018
@abraham abraham modified the milestone: v2.0 Jul 2, 2018
@abraham
Copy link
Owner

abraham commented Jul 4, 2018

This is going to be handled in a more generic way in #684

@abraham abraham closed this Jul 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants