Add AdSense Management API sample code to repository

[jcgregorio_google, 2011-10-25 17:40:25]

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

http://codereview.appspot.com/5310054/diff/1/samples/adsense/get_all_ad_clien...
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_cl...

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#...
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...

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

[sgomes, 2011-12-07 15:24:04]

Updated with suggested changes.

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

http://codereview.appspot.com/5310054/diff/1/samples/adsense/get_all_ad_clien...
samples/adsense/get_all_ad_clients.py:56: if 'nextPageToken' in result:
On 2011/10/25 17:40:25, jcgregorio_google wrote:
> Use list_next() pagination. See the Google+ sample:
> 
> 
>
http://code.google.com/p/google-plus-python-starter/source/browse/cli/plus_cl...
> 
> And:
> 
> http://code.google.com/p/google-api-python-client/wiki/Pagination
> 
> Applies to all the samples.

Done.

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#...
samples/adsense/sample_utils.py:34: # Set up a Flow object to be used if we need
to authenticate. This sample uses
On 2011/10/25 17:40:25, jcgregorio_google wrote:
> Please use the client_secrets.json support. See the Buzz sample:
> 
>
http://code.google.com/p/google-api-python-client/source/browse/#hg%252Fsampl...

Done.

http://codereview.appspot.com/5310054/diff/1/samples/adsense/sample_utils.py#...
samples/adsense/sample_utils.py:95: """Builds instance of service from local
discovery doc and does auth."""
On 2011/10/25 17:40:25, jcgregorio_google wrote:
> not local

Done.

[jcgregorio_google, 2011-12-07 16:26:28]

LGTM

On 2011/12/07 15:24:04, sgomes wrote:
> Updated with suggested changes.
> 
>
http://codereview.appspot.com/5310054/diff/1/samples/adsense/get_all_ad_clien...
> File samples/adsense/get_all_ad_clients.py (right):
> 
>
http://codereview.appspot.com/5310054/diff/1/samples/adsense/get_all_ad_clien...
> samples/adsense/get_all_ad_clients.py:56: if 'nextPageToken' in result:
> On 2011/10/25 17:40:25, jcgregorio_google wrote:
> > Use list_next() pagination. See the Google+ sample:
> > 
> > 
> >
>
http://code.google.com/p/google-plus-python-starter/source/browse/cli/plus_cl...
> > 
> > And:
> > 
> > http://code.google.com/p/google-api-python-client/wiki/Pagination
> > 
> > Applies to all the samples.
> 
> Done.
> 
> 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#...
> samples/adsense/sample_utils.py:34: # Set up a Flow object to be used if we
need
> to authenticate. This sample uses
> On 2011/10/25 17:40:25, jcgregorio_google wrote:
> > Please use the client_secrets.json support. See the Buzz sample:
> > 
> >
>
http://code.google.com/p/google-api-python-client/source/browse/#hg%25252Fsam...
> 
> Done.
> 
>
http://codereview.appspot.com/5310054/diff/1/samples/adsense/sample_utils.py#...
> samples/adsense/sample_utils.py:95: """Builds instance of service from local
> discovery doc and does auth."""
> On 2011/10/25 17:40:25, jcgregorio_google wrote:
> > not local
> 
> Done.

[jcgregorio_google, 2011-12-08 18:17:36]

Committed at
http://code.google.com/p/google-api-python-client/source/detail?r=e90ce87a9c5...

On 2011/12/07 16:26:28, jcgregorio_google wrote:
> LGTM
> 
> On 2011/12/07 15:24:04, sgomes wrote:
> > Updated with suggested changes.
> > 
> >
>
http://codereview.appspot.com/5310054/diff/1/samples/adsense/get_all_ad_clien...
> > File samples/adsense/get_all_ad_clients.py (right):
> > 
> >
>
http://codereview.appspot.com/5310054/diff/1/samples/adsense/get_all_ad_clien...
> > samples/adsense/get_all_ad_clients.py:56: if 'nextPageToken' in result:
> > On 2011/10/25 17:40:25, jcgregorio_google wrote:
> > > Use list_next() pagination. See the Google+ sample:
> > > 
> > > 
> > >
> >
>
http://code.google.com/p/google-plus-python-starter/source/browse/cli/plus_cl...
> > > 
> > > And:
> > > 
> > > http://code.google.com/p/google-api-python-client/wiki/Pagination
> > > 
> > > Applies to all the samples.
> > 
> > Done.
> > 
> > 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#...
> > samples/adsense/sample_utils.py:34: # Set up a Flow object to be used if we
> need
> > to authenticate. This sample uses
> > On 2011/10/25 17:40:25, jcgregorio_google wrote:
> > > Please use the client_secrets.json support. See the Buzz sample:
> > > 
> > >
> >
>
http://code.google.com/p/google-api-python-client/source/browse/#hg%2525252Fs...
> > 
> > Done.
> > 
> >
>
http://codereview.appspot.com/5310054/diff/1/samples/adsense/sample_utils.py#...
> > samples/adsense/sample_utils.py:95: """Builds instance of service from local
> > discovery doc and does auth."""
> > On 2011/10/25 17:40:25, jcgregorio_google wrote:
> > > not local
> > 
> > Done.