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

Issue 5303093: Add all GAN samples, bring them up to date

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 12 months ago by Luke Blanshard
Modified:
13 years, 11 months ago
Reviewers:
jcgregorio_google
CC:
apasha
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+647 lines, -56 lines) Patch
A samples/gan/README.txt View 1 chunk +4 lines, -0 lines 0 comments Download
A samples/gan/advertisers/advertisers.py View 1 chunk +189 lines, -0 lines 0 comments Download
A samples/gan/advertisers/advertisers_template.html View 1 chunk +85 lines, -0 lines 0 comments Download
M samples/gan/ccoffers/offers.py View 5 chunks +34 lines, -24 lines 0 comments Download
M samples/gan/ccoffers/offers_template.html View 4 chunks +28 lines, -10 lines 0 comments Download
A samples/gan/client_secrets.json View 1 chunk +9 lines, -0 lines 0 comments Download
M samples/gan/events/events.py View 6 chunks +37 lines, -22 lines 0 comments Download
A samples/gan/publishers/publishers.py View 1 chunk +188 lines, -0 lines 0 comments Download
A samples/gan/publishers/publishers_template.html View 1 chunk +73 lines, -0 lines 0 comments Download

Messages

Total messages: 4
jcgregorio_google
LGTM
13 years, 11 months ago (2011-11-11 17:22:11 UTC) #1
Luke Blanshard
Thanks Joe. I'm ignorant of the process for codesite, is the next step mine or ...
13 years, 11 months ago (2011-11-11 17:26:58 UTC) #2
jcgregorio_google
I just added you as a committer to the project, so you can commit that ...
13 years, 11 months ago (2011-11-11 17:29:51 UTC) #3
Luke Blanshard
13 years, 11 months ago (2011-11-11 17:32:54 UTC) #4
On Fri, Nov 11, 2011 at 11:29 AM, Joe Gregorio <jcgregorio@google.com>wrote: > I just added you as a committer to the project, so you can commit that > code. > Cool, thanks! Luke > > Thanks, > -joe > > On Fri, Nov 11, 2011 at 12:26 PM, Luke Blanshard <leadpipe@google.com> > wrote: > > Thanks Joe. I'm ignorant of the process for codesite, is the next step > mine > > or yours? > > Luke > > > > On Fri, Nov 11, 2011 at 11:22 AM, <jcgregorio@google.com> wrote: > >> > >> LGTM > >> > >> http://codereview.appspot.com/5303093/ > > > > > 
Sign in to reply to this message.

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