Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(11)

Issue 7852: Added paging support to the Contacts sample. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
17 years, 7 months ago by Trevor Johns (Google)
Modified:
16 years, 10 months ago
Reviewers:
j.s
CC:
gdata-python-client-library-contributors_googlegroups.com
Visibility:
Public.

Description

Added paging support to the Contacts sample. Updated Contacts API sample to support paging when working with accounts that have a large number of contacts and/or groups.

Patch Set 1 #

Patch Set 2 : Responded to Jeff's review comments. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+69 lines, -11 lines) Patch
M samples/contacts/contacts_example.py View 1 4 chunks +69 lines, -11 lines 0 comments Download

Messages

Total messages: 4
Trevor Johns (Google)
17 years, 7 months ago (2008-10-22 19:00:02 UTC) #1
j.s
http://codereview.appspot.com/7852/diff/1/2 File samples/contacts/contacts_example.py (right): http://codereview.appspot.com/7852/diff/1/2#newcode91 Line 91: next = atom.Link(href='http://www.google.com/m8/feeds/contacts/default/full') line too long http://codereview.appspot.com/7852/diff/1/2#newcode93 Line ...
17 years, 7 months ago (2008-10-22 21:09:48 UTC) #2
Trevor Johns (Google)
http://codereview.appspot.com/7852/diff/1/2 File samples/contacts/contacts_example.py (right): http://codereview.appspot.com/7852/diff/1/2#newcode91 Line 91: next = atom.Link(href='http://www.google.com/m8/feeds/contacts/default/full') On 2008/10/22 21:09:48, j.s wrote: ...
17 years, 7 months ago (2008-10-22 23:18:15 UTC) #3
j.s
17 years, 7 months ago (2008-10-23 00:10:28 UTC) #4
LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b