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

Issue 3863041: Sample code for Shopping API usage.

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 3 months ago by aherrman
Modified:
13 years, 3 months ago
Reviewers:
jcgregorio, Ali Afshar
CC:
google-api-python-client_googlegroups.com
Visibility:
Public.

Patch Set 1 #

Total comments: 2

Patch Set 2 : Removed path modifications. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -9 lines) Patch
M samples/shopping/main.py View 1 1 chunk +41 lines, -9 lines 0 comments Download

Messages

Total messages: 7
aherrman
13 years, 3 months ago (2011-01-04 14:14:29 UTC) #1
jcgregorio
http://codereview.appspot.com/3863041/diff/1/samples/shopping/main.py File samples/shopping/main.py (right): http://codereview.appspot.com/3863041/diff/1/samples/shopping/main.py#newcode17 samples/shopping/main.py:17: sys.path.insert(0, os.getcwd()) Please don't do this, the common way ...
13 years, 3 months ago (2011-01-04 14:33:12 UTC) #2
aherrman
http://codereview.appspot.com/3863041/diff/1/samples/shopping/main.py File samples/shopping/main.py (right): http://codereview.appspot.com/3863041/diff/1/samples/shopping/main.py#newcode17 samples/shopping/main.py:17: sys.path.insert(0, os.getcwd()) I thought I grabbed that from another ...
13 years, 3 months ago (2011-01-04 14:38:16 UTC) #3
Ali Afshar
Hi, not sure if you guys saw these: http://codereview.appspot.com/3853041/ Relating to the same issue. On ...
13 years, 3 months ago (2011-01-04 14:38:38 UTC) #4
aherrman
I hadn't seen those. They look more comprehensive on the query side. The only thing ...
13 years, 3 months ago (2011-01-04 14:43:01 UTC) #5
aherrman
13 years, 3 months ago (2011-01-04 15:29:12 UTC) #6
jcgregorio
13 years, 3 months ago (2011-01-04 15:39:13 UTC) #7
LGTM
Sign in to reply to this message.

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