-
Notifications
You must be signed in to change notification settings - Fork 262
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
HandleKeywordPolicyViolations not getting the right exception #1012
Labels
Comments
There might be some changes recently that we aren't aware of. Let me look into this and will update this thread when I have more clues. |
Did you use REST, not gRPC? |
Yes, I use REST
Envoyé à partir de Outlook pour iOS<https://aka.ms/o0ukef>
…________________________________
De : Thanet Knack Praneenararat ***@***.***>
Envoyé : Tuesday, June 4, 2024 5:32:35 PM
À : googleads/google-ads-php ***@***.***>
Cc : SeriousJ55 ***@***.***>; Author ***@***.***>
Objet : Re: [googleads/google-ads-php] HandleKeywordPolicyViolations not getting the right exception (Issue #1012)
Did you use REST, not gRPC?
—
Reply to this email directly, view it on GitHub<#1012 (comment)>, or unsubscribe<https:/notifications/unsubscribe-auth/AGNJVK6ONZ7HALWYEK3HVO3ZFXMZHAVCNFSM6AAAAABGXG4KXWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBXHAZTENRWGE>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Ah. OK. REST cannot handle errors as well as gRPC right now. We have this in our feature request list, but it's going to take some time to finish it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The code in examples/ErrorHandling/HandleKeywordPolicyViolations.php doesn't work because policy violations of a keyword are thrown as ApiExceptions and not as GoogleAdsExceptions.
For instance, here are the first lines an error received for a policy violation:
As you can see, this is an ApiException. So I can't access the associated policyViolationDetailsKey and thus can't ask for an exemption.
What am I missing?
The text was updated successfully, but these errors were encountered: