Update adexchangebuyer samples to use apiclient.sample_tools.

Update adexchangebuyer samples to use apiclient.sample_tools.

[Joseph DiLallo, 2013-04-05 18:13:36]

Thanks for updating these. I have just 1 minor questions about the argparse
declarations...

https://codereview.appspot.com/8410044/diff/1/samples/adexchangebuyer/submit_...
File samples/adexchangebuyer/submit_creative.py (right):

https://codereview.appspot.com/8410044/diff/1/samples/adexchangebuyer/submit_...
samples/adexchangebuyer/submit_creative.py:39:
argparser.add_argument('agency_id',
We were using flags.DEFINE_string, but this really should be an int too. (It's a
long in the API). Should we make add type=int for consistency?

(Buyer creative ID is correctly a string)

https://codereview.appspot.com/8410044/diff/1/samples/adexchangebuyer/update_...
File samples/adexchangebuyer/update_account.py (right):

https://codereview.appspot.com/8410044/diff/1/samples/adexchangebuyer/update_...
samples/adexchangebuyer/update_account.py:33:
argparser.add_argument('account_id',
I notice this is the only file where the account_id argument is missing type=int

[jcgregorio_google, 2013-04-05 18:18:15]

https://codereview.appspot.com/8410044/diff/1/samples/adexchangebuyer/submit_...
File samples/adexchangebuyer/submit_creative.py (right):

https://codereview.appspot.com/8410044/diff/1/samples/adexchangebuyer/submit_...
samples/adexchangebuyer/submit_creative.py:39:
argparser.add_argument('agency_id',
On 2013/04/05 18:13:36, Joseph DiLallo wrote:
> We were using flags.DEFINE_string, but this really should be an int too. (It's
a
> long in the API). Should we make add type=int for consistency?
> 
> (Buyer creative ID is correctly a string)

Done.

https://codereview.appspot.com/8410044/diff/1/samples/adexchangebuyer/update_...
File samples/adexchangebuyer/update_account.py (right):

https://codereview.appspot.com/8410044/diff/1/samples/adexchangebuyer/update_...
samples/adexchangebuyer/update_account.py:33:
argparser.add_argument('account_id',
On 2013/04/05 18:13:36, Joseph DiLallo wrote:
> I notice this is the only file where the account_id argument is missing
type=int

Done.

[Joseph DiLallo, 2013-04-05 18:19:03]

LGTM

[jcgregorio_google, 2013-04-05 18:26:50]

Commited in
https://code.google.com/p/google-api-python-client/source/detail?r=34b8b62795...

On Fri, Apr 5, 2013 at 2:19 PM,  <jdilallo@google.com> wrote:
> LGTM
>
> https://codereview.appspot.com/8410044/

[jcgregorio_google, 2013-04-05 18:26:56]

On 2013/04/05 18:19:03, Joseph DiLallo wrote:
> LGTM

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