Closed GoogleCodeExporter closed 9 years ago
See issue 115 for ranking scheme..
Original comment by diab...@gmail.com
on 12 May 2012 at 7:20
Original comment by diab...@gmail.com
on 12 May 2012 at 1:14
This issue was updated by revision r1113.
adding tests
Original comment by diab...@gmail.com
on 12 May 2012 at 9:55
Original comment by diab...@gmail.com
on 12 May 2012 at 10:42
Original comment by diab...@gmail.com
on 13 May 2012 at 9:17
Leh keda? Makanet kwayesa!
Original comment by guc@sakr.me
on 13 May 2012 at 9:20
I thought I should move the tests from integration tests to functional
tests the ones regarding the view ama ba check div aw list
shouldn't I ?
Original comment by diab...@gmail.com
on 13 May 2012 at 9:22
Original comment by diab...@gmail.com
on 13 May 2012 at 9:32
This issue was updated by revision r1288.
adding backend methods with doc
Original comment by diab...@gmail.com
on 13 May 2012 at 8:01
This issue was updated by revision r1297.
adding pie chart to the all interests statistics view
Original comment by diab...@gmail.com
on 13 May 2012 at 9:53
This issue was updated by revision r1431.
editing working tests , removing RED
Original comment by diab...@gmail.com
on 15 May 2012 at 4:19
This issue was updated by revision r1432.
adding removed associations , fixing calling the graphs in the view
Original comment by diab...@gmail.com
on 15 May 2012 at 4:24
This issue was updated by revision r1445.
adding method to get top ranked interests and listing them in the view
Original comment by diab...@gmail.com
on 15 May 2012 at 5:37
updated Scenarios
Success scenarios:
"A comparison between top 5 interests are done as a pie chart showing interests
that are added or having stories the most.A list of top interests and there
ranks will appear and on clicking on one of them the admin will be redirected
to the statistics page of the statistics of this specific interest over the
last 30 days. "
Failure scenarios:
1- There are no interest in the system created to show the top interests in
this case an error message will appear.
2- no interest could be ranked because all of them don't have any users or
stories so an error message will appear.
Original comment by diab...@gmail.com
on 15 May 2012 at 6:15
This issue was updated by revision r1452.
editing view
Original comment by diab...@gmail.com
on 15 May 2012 at 6:18
This issue was updated by revision r1469.
adding UML
Original comment by diab...@gmail.com
on 15 May 2012 at 7:52
Original comment by diab...@gmail.com
on 15 May 2012 at 7:53
This issue was updated by revision r1536.
editing style (components instead of a table)
Original comment by diab...@gmail.com
on 16 May 2012 at 12:56
This issue was updated by revision r1538.
editing style
Original comment by diab...@gmail.com
on 16 May 2012 at 1:26
This issue was updated by revision r1539.
editing style
Original comment by diab...@gmail.com
on 16 May 2012 at 1:32
Original comment by mohamed....@gmail.com
on 18 May 2012 at 9:18
Original comment by mohamed....@gmail.com
on 18 May 2012 at 9:18
This issue was updated by revision r1709.
moving view tests to functional tests
Original comment by diab...@gmail.com
on 18 May 2012 at 11:09
Original comment by diab...@gmail.com
on 18 May 2012 at 4:22
see r1445 and other revisions indenting your code and return it back to
code-ready
Original comment by christin...@gmail.com
on 19 May 2012 at 12:43
see r1539
Original comment by christin...@gmail.com
on 19 May 2012 at 12:46
Please check the comments on the UML in revision 1469..
Removing the label UML-Ready after commenting..
Original comment by lydia.si...@gmail.com
on 19 May 2012 at 2:33
Documentation should be in the format:
= begin
Method description
Parameters (inputs, outputs)
Author: Diab
= end
In addition to simple in line documentation to inside the methods if possible
Original comment by mohamed....@gmail.com
on 19 May 2012 at 2:38
-(Nothing But a Design Issue ) When there is no users adding this certain
Interest , why is it in a form of a Red flash "Warning" ? ... maybe a
notifying blue/orange message would be nice .
-What if ? ... this Interest is blocked ? ... i think it would be better if
there is a label beside each interest indicating whether it's blocked or not .
Original comment by jailan.s...@gmail.com
on 20 May 2012 at 11:24
This issue was updated by revision r1962.
adding UML
Original comment by diab...@gmail.com
on 20 May 2012 at 12:53
the format =begin , =end
causes a compiler error with me , u sure of this ?
Original comment by diab...@gmail.com
on 20 May 2012 at 1:19
if you checked you're gonna find out that my docs matches this format except
for the begin end thing
Original comment by diab...@gmail.com
on 20 May 2012 at 1:27
This issue was updated by revision r1979.
fixing reviewers comments
Original comment by diab...@gmail.com
on 20 May 2012 at 2:28
This issue was updated by revision r1987.
adding labels indicating the status of the interest in the list
Original comment by diab...@gmail.com
on 20 May 2012 at 3:18
This issue was updated by revision r1992.
updating tests for the new authentication system
Original comment by diab...@gmail.com
on 20 May 2012 at 4:05
Original comment by mohamed....@gmail.com
on 20 May 2012 at 4:25
Original comment by lydia.si...@gmail.com
on 20 May 2012 at 7:07
This issue was updated by revision r2022.
editing UML
Original comment by diab...@gmail.com
on 20 May 2012 at 7:53
Original comment by christin...@gmail.com
on 20 May 2012 at 8:24
Original comment by lydia.si...@gmail.com
on 20 May 2012 at 9:38
can the div with id ccc have a more descriptive name?!
Original comment by mostafa....@gmail.com
on 20 May 2012 at 10:25
There are no Tests for failure scenarios
Original comment by mostafa....@gmail.com
on 20 May 2012 at 11:22
This issue was updated by revision r2159.
moving some tests and making them pass
Original comment by diab...@gmail.com
on 23 May 2012 at 12:04
This issue was updated by revision r2222.
adding test for failure scenario , and changing the chart div id to a
meaningful name
Original comment by diab...@gmail.com
on 23 May 2012 at 12:29
Original comment by jailan.s...@gmail.com
on 23 May 2012 at 8:38
Original comment by mostafa....@gmail.com
on 23 May 2012 at 8:50
Original comment by yahiaelg...@gmail.com
on 23 May 2012 at 11:13
Original issue reported on code.google.com by
diab...@gmail.com
on 10 May 2012 at 9:17