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

Issue 5442046: Update ReadMe.txt and fix scope of overridden methods.

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 5 months ago by jjc
Modified:
2 years ago
Reviewers:
nrhodes, fmela
CC:
analytics-api-samples-discuss_googlegroups.com
Base URL:
http://analytics-api-samples.googlecode.com/svn/trunk/src/tracking/mobile/android/EasyTracker/Library/
Visibility:
Public.

Patch Set 1 #

Total comments: 4

Patch Set 2 : Fix ReadMe.txt #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -6 lines) Patch
M ReadMe.txt View 1 1 chunk +10 lines, -0 lines 0 comments Download
M src/com/google/android/apps/analytics/easytracking/TrackedActivity.java View 3 chunks +3 lines, -3 lines 0 comments Download
M src/com/google/android/apps/analytics/easytracking/TrackedListActivity.java View 3 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 6
jjc
12 years, 5 months ago (2011-11-28 17:54:00 UTC) #1
nrhodes
http://codereview.appspot.com/5442046/diff/1/ReadMe.txt File ReadMe.txt (right): http://codereview.appspot.com/5442046/diff/1/ReadMe.txt#newcode44 ReadMe.txt:44: Remember to add the file libGoogleAnalytics.jar to your application's ...
12 years, 5 months ago (2011-11-28 17:57:40 UTC) #2
jjc
12 years, 5 months ago (2011-11-28 18:01:59 UTC) #3
jjc
http://codereview.appspot.com/5442046/diff/1/ReadMe.txt File ReadMe.txt (right): http://codereview.appspot.com/5442046/diff/1/ReadMe.txt#newcode44 ReadMe.txt:44: Remember to add the file libGoogleAnalytics.jar to your application's ...
12 years, 5 months ago (2011-11-28 18:02:06 UTC) #4
nrhodes
LGTM
12 years, 5 months ago (2011-11-28 18:06:55 UTC) #5
fmela
12 years, 5 months ago (2011-11-28 21:06:46 UTC) #6
LGTM
Sign in to reply to this message.

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