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

You need to add exception handler for searchterm #3

Open
HamoudAQ opened this issue Aug 17, 2020 · 2 comments
Open

You need to add exception handler for searchterm #3

HamoudAQ opened this issue Aug 17, 2020 · 2 comments

Comments

@HamoudAQ
Copy link

I noticed that if I pass a campaign that is new then it would break, as this is the response from apple:
{'data': {'reportingDataResponse': {'row': []}}, 'pagination': {'totalResults': 0, 'startIndex': 0, 'itemsPerPage': 0}, 'error': None}
and obviously there's no grandTotals which is mentioned in the apy.py, line 1429
res["data"]["reportingDataResponse"]["grandTotals"])

a quick solution is to add try and except

@elettradamaggio-gc
Copy link

That's a good point I actually have the same issue. anyone coded a fix?

@alkattan
Copy link
Collaborator

Hi @HamoudAQ,
Could you please create a pull request with the change you've asked for?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants