vacation responder sample

Temporarily put there for review. Blogpost in review, so just wanted to get this done too.

Thank you
Shraddha

[Ali Afshar, 2011-12-20 17:38:05]

Great sample, only minor issues.

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py
File samples/vacation_responder.py (right):

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:60: SCOPE =
('https://www.googleapis.com/auth/calendar '
Add a comment here to say that we are requesting a token for the scope of two
separate APIs

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:77: 
remove blank line

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:87: 
remove blank line

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:105: 
remove blank line

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:133: 
remove blank line

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:152: print 'invalid credentials'
This is not exactly try, the credentials may not exist. Perhaps "Credentials are
invalid or don't exist."

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:160: # Get OAuth 2.0 token to be used with gdata
client library
Create an OAuth2.0 token suitable for use with the GData client library.

[shraddhag, 2011-12-21 10:54:02]

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py
File samples/vacation_responder.py (right):

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:60: SCOPE =
('https://www.googleapis.com/auth/calendar '
On 2011/12/20 17:38:05, Ali Afshar wrote:
> Add a comment here to say that we are requesting a token for the scope of two
> separate APIs

Done.

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:77: 
On 2011/12/20 17:38:05, Ali Afshar wrote:
> remove blank line

Done.

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:87: 
On 2011/12/20 17:38:05, Ali Afshar wrote:
> remove blank line

Done.

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:105: 
On 2011/12/20 17:38:05, Ali Afshar wrote:
> remove blank line

Done.

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:133: 
On 2011/12/20 17:38:05, Ali Afshar wrote:
> remove blank line

Done.

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:152: print 'invalid credentials'
On 2011/12/20 17:38:05, Ali Afshar wrote:
> This is not exactly try, the credentials may not exist. Perhaps "Credentials
are
> invalid or don't exist."

Done.

http://codereview.appspot.com/5490079/diff/1/samples/vacation_responder.py#ne...
samples/vacation_responder.py:160: # Get OAuth 2.0 token to be used with gdata
client library
On 2011/12/20 17:38:05, Ali Afshar wrote:
> Create an OAuth2.0 token suitable for use with the GData client library.

Done.

[Ali Afshar, 2011-12-21 11:04:22]

LGTM