Re: Review Request 40: basic RESTful API

2013-08-16 Thread Ilgiz Islamgulov
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/40/ --- (Updated Aug. 16, 2013, 1:38 p.m.) Status -- This change has been

Re: Review Request 40: basic RESTful API

2013-08-15 Thread Tim Flink
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/40/#review68 --- Ship it! Overall, what I'm seeing looks good. Can you change

Re: Review Request 40: basic RESTful API

2013-08-06 Thread Tim Flink
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/40/#review62 --- blockerbugs.spec

Re: Review Request 40: basic RESTful API

2013-08-05 Thread Ilgiz Islamgulov
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/40/#review60 --- - Ilgiz Islamgulov On July 22, 2013, 12:04 p.m., Ilgiz Islamg

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 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

Review Request 40: basic RESTful API

2013-07-22 Thread Ilgiz Islamgulov
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/40/ --- Review request for blockerbugs. Bugs: 392 https://fedorahosted.org/