Update adsense samples to use apliclient.sample_tools.

Update adsense samples to use apliclient.sample_tools.

[dhermes, 2013-04-05 17:24:59]

You and Sergio good on this review or do you need my eyes too?

[jcgregorio_google, 2013-04-05 17:26:52]

No, I added you so you could see apiclient.sample_tools in action.

On Fri, Apr 5, 2013 at 1:24 PM,  <dhermes@google.com> wrote:
> You and Sergio good on this review or do you need my eyes too?
>
> https://codereview.appspot.com/8249044/

[dhermes, 2013-04-05 18:36:02]

General changes look good.

I'll leave it to Sergio for the final OK

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/generate_rep...
File samples/adsense/generate_report.py (right):

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/generate_rep...
samples/adsense/generate_report.py:35: help='The ID of the ad client for which
to generate a report')
What happened to the short name?

[jcgregorio_google, 2013-04-05 18:39:08]

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/generate_rep...
File samples/adsense/generate_report.py (right):

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/generate_rep...
samples/adsense/generate_report.py:35: help='The ID of the ad client for which
to generate a report')
The short name is gone because these are now positional cmd-line parameters.

On 2013/04/05 18:36:02, dhermes wrote:
> What happened to the short name?

[jalc, 2013-04-26 17:09:57]

Minor comments, LGTM.

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
File samples/adsense/get_all_custom_channels.py (right):

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
samples/adsense/get_all_custom_channels.py:35: help='The ID of the ad client for
which to generate a report')
The ad client ID for which to get custom channels

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
File samples/adsense/get_all_custom_channels_for_ad_unit.py (right):

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
samples/adsense/get_all_custom_channels_for_ad_unit.py:40: help='The ID of the
ad client for which to generate a report')
"The ID of the ad client with the specified ad unit"

It makes more sense to have the account before the ad client.

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_url_...
File samples/adsense/get_all_url_channels.py (right):

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_url_...
samples/adsense/get_all_url_channels.py:37: help='The ID of the ad client for
which to generate a report')
The ad client ID for which to get URL channels

[jcgregorio_google, 2013-05-16 20:54:51]

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
File samples/adsense/get_all_custom_channels.py (right):

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
samples/adsense/get_all_custom_channels.py:35: help='The ID of the ad client for
which to generate a report')
On 2013/04/26 17:09:57, jalc wrote:
> The ad client ID for which to get custom channels

Done.

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
File samples/adsense/get_all_custom_channels_for_ad_unit.py (right):

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
samples/adsense/get_all_custom_channels_for_ad_unit.py:40: help='The ID of the
ad client for which to generate a report')
On 2013/04/26 17:09:57, jalc wrote:
> "The ID of the ad client with the specified ad unit"
> 
> It makes more sense to have the account before the ad client.

Done.

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_url_...
File samples/adsense/get_all_url_channels.py (right):

https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_url_...
samples/adsense/get_all_url_channels.py:37: help='The ID of the ad client for
which to generate a report')
On 2013/04/26 17:09:57, jalc wrote:
> The ad client ID for which to get URL channels

Done.

[jcgregorio_google, 2013-05-20 20:06:10]

Committed in
https://code.google.com/p/google-api-python-client/source/detail?r=f40bfee6a4...

On 2013/05/16 20:54:51, jcgregorio_google wrote:
>
https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
> File samples/adsense/get_all_custom_channels.py (right):
> 
>
https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
> samples/adsense/get_all_custom_channels.py:35: help='The ID of the ad client
for
> which to generate a report')
> On 2013/04/26 17:09:57, jalc wrote:
> > The ad client ID for which to get custom channels
> 
> Done.
> 
>
https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
> File samples/adsense/get_all_custom_channels_for_ad_unit.py (right):
> 
>
https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_cust...
> samples/adsense/get_all_custom_channels_for_ad_unit.py:40: help='The ID of the
> ad client for which to generate a report')
> On 2013/04/26 17:09:57, jalc wrote:
> > "The ID of the ad client with the specified ad unit"
> > 
> > It makes more sense to have the account before the ad client.
> 
> Done.
> 
>
https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_url_...
> File samples/adsense/get_all_url_channels.py (right):
> 
>
https://codereview.appspot.com/8249044/diff/5001/samples/adsense/get_all_url_...
> samples/adsense/get_all_url_channels.py:37: help='The ID of the ad client for
> which to generate a report')
> On 2013/04/26 17:09:57, jalc wrote:
> > The ad client ID for which to get URL channels
> 
> Done.