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

Issue 9971043: Add an account setting to opt-in to new UI

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 1 month ago by rch
Modified:
5 hours, 44 minutes ago
CC:
codereview-discuss_googlegroups.com
Visibility:
Public.

Description

Add an account setting to opt-in to new UI features.

Patch Set 1 #

Total comments: 2

Patch Set 2 : Fix comments... #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+178 lines, -91 lines) Patch
M codereview/models.py View 1 chunk +2 lines, -0 lines 0 comments Download
M codereview/views.py View 5 chunks +7 lines, -0 lines 0 comments Download
M static/styles.css View 1 chunk +0 lines, -4 lines 0 comments Download
A static/styles-experimental.css View 1 chunk +3 lines, -0 lines 0 comments Download
M templates/base.html View 1 chunk +3 lines, -0 lines 0 comments Download
M templates/user.html View 1 1 chunk +3 lines, -87 lines 0 comments Download
A templates/user_experimental_include.html View 1 1 chunk +89 lines, -0 lines 1 comment Download
A templates/user_include.html View 1 1 chunk +71 lines, -0 lines 0 comments Download

Messages

Total messages: 20
rch
How does this look?
12 years, 1 month ago (2013-06-03 16:53:44 UTC) #1
iannucci
I'd add the codereview-discuss@googlegroups.com to the CC list and M-A to reviewers since this will ...
12 years, 1 month ago (2013-06-03 19:03:40 UTC) #2
rch
I've addressed your comment and have not seen other comments. When would you like to ...
12 years, 1 month ago (2013-06-06 19:38:58 UTC) #3
iannucci
+codereview-discuss@googlegroups.com I didn't see the discuss group on the CC? Anyhow, this LGTM. I'll land ...
12 years, 1 month ago (2013-06-07 03:19:56 UTC) #4
iannucci
+maruel I'd like to get an LGTM on this from an owner :)
12 years, 1 month ago (2013-06-07 17:40:42 UTC) #5
M-A Ruel
On 2013/06/07 17:40:42, iannucci wrote: > +maruel I'd like to get an LGTM on this ...
12 years, 1 month ago (2013-06-07 17:42:42 UTC) #6
iannucci
On 2013/06/07 17:42:42, M-A Ruel wrote: > On 2013/06/07 17:40:42, iannucci wrote: > > +maruel ...
12 years, 1 month ago (2013-06-07 17:49:19 UTC) #7
M-A Ruel
On 2013/06/07 17:49:19, iannucci wrote: > On 2013/06/07 17:42:42, M-A Ruel wrote: > > On ...
12 years, 1 month ago (2013-06-07 18:02:54 UTC) #8
iannucci
On 2013/06/07 18:02:54, M-A Ruel wrote: > On 2013/06/07 17:49:19, iannucci wrote: > > On ...
12 years, 1 month ago (2013-06-07 18:33:20 UTC) #9
anagy4698
2013/6/7 <maruel@google.com>: > On 2013/06/07 17:49:19, iannucci wrote: >> >> On 2013/06/07 17:42:42, M-A Ruel ...
12 years, 1 month ago (2013-06-07 19:15:19 UTC) #10
rch
On 2013/06/07 18:33:20, iannucci wrote: > On 2013/06/07 18:02:54, M-A Ruel wrote: > > On ...
12 years ago (2013-06-17 23:49:25 UTC) #11
M-A
On 2013/06/07 18:33:20, iannucci wrote: > On 2013/06/07 18:02:54, M-A Ruel wrote: > > That's ...
12 years ago (2013-06-17 23:54:44 UTC) #12
iannucci
On 2013/06/17 23:54:44, M-A wrote: > On 2013/06/07 18:33:20, iannucci wrote: > > On 2013/06/07 ...
12 years ago (2013-06-17 23:57:47 UTC) #13
anagy4698
2013/6/18 <rch@chromium.org> > On 2013/06/07 18:33:20, iannucci wrote: > >> On 2013/06/07 18:02:54, M-A Ruel ...
12 years ago (2013-06-18 09:46:40 UTC) #14
Andi
Sorry, I didn't followed the whole discussion. What's the long-term purpose of this patch? For ...
12 years ago (2013-06-18 19:13:57 UTC) #15
iannucci
On 2013/06/18 19:13:57, Andi wrote: > Sorry, I didn't followed the whole discussion. What's the ...
12 years ago (2013-06-18 19:18:47 UTC) #16
Andi
On Tue, Jun 18, 2013 at 9:18 PM, <iannucci@chromium.org> wrote: > On 2013/06/18 19:13:57, Andi ...
12 years ago (2013-06-18 19:34:34 UTC) #17
M-A Ruel
On my side, I think it's more a naming issue; it's not experimental if it's ...
12 years ago (2013-06-20 14:34:26 UTC) #18
sec.test.dragnet
5 hours, 44 minutes ago (2025-07-07 19:31:56 UTC) #19
sec.test.dragnet
5 hours, 44 minutes ago (2025-07-07 19:31:57 UTC) #20
irrelevant
Sign in to reply to this message.

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