Little change to idea for KDE Reports

Ben Cooksley bcooksley at
Mon Mar 17 06:27:34 UTC 2014

On Sun, Mar 16, 2014 at 1:24 PM, Yash Ladia <ladiayash at> wrote:
> Hello,

Hi Yash,

> Firstly, thank-you for replying to my emails so promptly.
> While I was writing my proposal, I felt it would be nice to change the idea
> a bit:
> According to the text on the idea page, the idea has the following parts:
>   1. Completing the test suite
>   2. upgrading to rails 4
>   3. Adding other activities.
> Though adding more features to our site is nice, I think it is more
> important to change the code so that adding more activities is easier and
> done with least code duplication.

This makes sense.

> Hence, I would like to refract the idea to:
>   1. Completing the test suite (same as above)
>   2. Cleaning up and modularizing the existing testing code:
> Like I had done in my review request [1], I would make similar changes to
> other parts of the integration tests code.
>   3. Creating a generic base module for all activities:
> All the activities will have some very common parts (entries, xml parsing,
> fetching of the xml file, etc.) I will create a base module "activity". This

The XML file is rather specific to Git repository handling - as
previously stated the forum, mailing lists, news feeds, etc. won't
have this data available.

> would be implemented by all activities (both that exist now and the ones
> that will be added in the future). I chose module over class, as different
> activities may have some differences in their tables which would lead to
> denormalization in case of classes [2].
>   4. Porting the projects activity backend code to the above structure.
>   5. Create the backend code for the news activity (accordingly):
> This will be done if time permits, but will not take as much time as a full
> new activity since the integration tests are already written.
> [1]
> [2]
> Please provide your thoughts on the above so that I can change my proposal
> accordingly.
> Regards,
> Yash.


More information about the kde-www mailing list