Add AdSense Management API sample code to repository (issue 5310054)

6 views
Skip to first unread message

jcgre...@google.com

unread,
Oct 25, 2011, 1:40:25 PM10/25/11
to sergio...@google.com, google-api-p...@googlegroups.com, re...@codereview.appspotmail.com

http://codereview.appspot.com/5310054/diff/1/samples/adsense/get_all_ad_clients.py
File samples/adsense/get_all_ad_clients.py (right):

http://codereview.appspot.com/5310054/diff/1/samples/adsense/get_all_ad_clients.py#newcode56
samples/adsense/get_all_ad_clients.py:56: if 'nextPageToken' in result:
Use list_next() pagination. See the Google+ sample:


http://code.google.com/p/google-plus-python-starter/source/browse/cli/plus_cli.py#75

And:

http://code.google.com/p/google-api-python-client/wiki/Pagination

Applies to all the samples.

http://codereview.appspot.com/5310054/diff/1/samples/adsense/sample_utils.py
File samples/adsense/sample_utils.py (right):

http://codereview.appspot.com/5310054/diff/1/samples/adsense/sample_utils.py#newcode34
samples/adsense/sample_utils.py:34: # Set up a Flow object to be used if
we need to authenticate. This sample uses
Please use the client_secrets.json support. See the Buzz sample:

http://code.google.com/p/google-api-python-client/source/browse/#hg%2Fsamples%2Fbuzz

http://codereview.appspot.com/5310054/diff/1/samples/adsense/sample_utils.py#newcode95
samples/adsense/sample_utils.py:95: """Builds instance of service from
local discovery doc and does auth."""
not local

http://codereview.appspot.com/5310054/

Reply all
Reply to author
Forward
0 new messages