Skip to content
This repository was archived by the owner on Jun 8, 2023. It is now read-only.

Bing v13 API upgrade - #17

Open
ssrva wants to merge 28 commits into
FindHotel:masterfrom
clarisights:master
Open

Bing v13 API upgrade#17
ssrva wants to merge 28 commits into
FindHotel:masterfrom
clarisights:master

Conversation

@ssrva

@ssrva ssrva commented Oct 24, 2019

Copy link
Copy Markdown
  • Adding support for bing api v13.
  • Deprecating v11.
  • Using camel case instead of classify in ReportRequest
  • Handling nil in ReportingService (report_body operation)

anantprsd5 and others added 22 commits February 24, 2020 12:16
Add format_version as a valid parameter in report_request
Make it same as that of adwyze
fix(int, bing): categorize CallRateExceeded error as rate limit error
fix(bing): fix syntax error and version upgrade
Centralizes retry mechanisms and introduces dedicated retry handlers
fix(int, bing): Improve SOAP client error handling and retry logic
I missed to update the version last time we made some changes to this repo. [history](https://github.com/clarisights/bing-ads/commits/master/)
Updating the version so that adwyze code can pick it up once we update the
version there.
Intended changes were mostly about error handling on ratelimit errors.
chore(deps): relax active support dependency
Add AccountNotAuthorized error class and map operation_error code
'AccountNotAuthorized' to it in handle_soap_fault. This allows callers
to distinguish account privilege errors from generic SOAP faults,
enabling proper user-failure classification.

Bump version: 13.0.4 -> 13.0.5
…for-770

fix: Treat UserIsNotAuthorized as a typed exception
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.