Re: Review Request 36: improve update syncs

2013-08-02 Thread Ilgiz Islamgulov
> On Aug. 2, 2013, 2:58 p.m., Tim Flink wrote: > > blockerbugs/util/update_sync.py, line 40 > > > > > > I don't understand the reason for this change - it seems purely > > stylistic yes, it's stylistic only changes.

Re: Review Request 40: basic RESTful API

2013-08-02 Thread Ilgiz Islamgulov
> On Aug. 2, 2013, 2:38 p.m., Tim Flink wrote: > > blockerbugs/controllers/api/iso8601.py, line 48 > > > > > > why are you duplicating this code instead of pulling it in as a > > dependency? Isn't it the same code as

Re: Review Request 41: Login cleanup + redirect fix

2013-08-02 Thread Tim Flink
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/41/#review58 --- Ship it! Looks good to me - Tim Flink On Aug. 1, 2013, 9:26

Re: Review Request 36: improve update syncs

2013-08-02 Thread Tim Flink
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/36/#review57 --- blockerbugs/controllers/main.py

Re: Review Request 40: basic RESTful API

2013-08-02 Thread Tim Flink
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/40/#review56 --- There are GPL headers missing from most of the new source files