Bug#756766: Initial work-patch on this bug

2015-11-11 Thread Orestis Ioannou
Hey, On 11/04/2015 10:04 PM, Raphael Hertzog wrote: > Hello Orestis, > > sorry that I did not find the time to get back to you earlier. > no worries ;) > 1/ that the package page contains a link to the package news page (the > link you added in the header of the panel) > 2/ that the package

Bug#756766: Initial work-patch on this bug

2015-11-04 Thread Raphael Hertzog
Hello Orestis, sorry that I did not find the time to get back to you earlier. On Mon, 19 Oct 2015, Orestis Ioannou wrote: > I reworked most of the patch, using the ListView and factorising most of > the code. The code looks clean and slick. Good work! > Let me know if this needs more work :)

Bug#756766: Initial work-patch on this bug

2015-10-18 Thread Orestis Ioannou
Hello, I reworked most of the patch, using the ListView and factorising most of the code. Let me know if this needs more work :) Cheers, Orestis From 604f9981bc8630b3a21fb55ca5c35780d3094f40 Mon Sep 17 00:00:00 2001 From: Orestis Ioannou Date: Wed, 2 Sep 2015 10:38:10

Bug#756766: Initial work-patch on this bug

2015-09-12 Thread Raphael Hertzog
On Mon, 07 Sep 2015, Orestis Ioannou wrote: > Thanks for the review. > I updated my patch Thanks. Here are my comments. > > And then you reuse that code for functional testing with selenium (cf > > functional_tests/tests.py). > > Added a script. I am not sure though if the directory i put it is

Bug#756766: Initial work-patch on this bug

2015-09-07 Thread Orestis Ioannou
Hey, Thanks for the review. I updated my patch On Thu, 3 Sep 2015 21:00:14 +0200 Raphael Hertzog wrote: > > If you want to test this, you have to add many news to a single package. > You can do that manually I guess (either in an interactive Python shell > or with the help

Bug#756766: Initial work-patch on this bug

2015-09-03 Thread Raphael Hertzog
Hi, On Wed, 02 Sep 2015, Orestis Ioannou wrote: > I have enabled only the unstable repository and I don't have many news > to test what i ve done. Is there any other solution to test/work this > besides adding more repositories? s/repositories/news/ ? If you want to test this, you have to add

Bug#756766: Initial work-patch on this bug

2015-09-02 Thread Orestis Ioannou
Hello, I started working on this bug. I have a working patch but it requires still some work. I am not sure how to do the js part so for the moment i concentrated on the non js requirements. I have a question though: I have enabled only the unstable repository and I don't have many news to test