zeitgeist team mailing list archive

Thread
Date

Re: [Merge] lp:~cando/activity-log-manager/925638 into lp:activity-log-manager

To: Stefano Candori <stefano.candori@xxxxxxxxx>
From: Seif Lotfy <seif@xxxxxxxxx>
Date: Fri, 24 Feb 2012 14:52:47 -0000
In-reply-to: <20120224134418.29193.10560.launchpad@ackee.canonical.com>
Reply-to: mp+94543@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Nice Stefano...
Feel free to merge it..
While you are at it can you fix "handle_app_population". IMHO it should not be called from inside the "find_events"
Rather have it called up from the construct with all installed applications, sorted alphabetically.
Then for each app asynchronosly ask zeitgeist for "HOW MANY EVENTS" it has for it. This way it is more accurate.

-- 
https://code.launchpad.net/~cando/activity-log-manager/925638/+merge/94543
Your team Activity Log Manager is requested to review the proposed merge of lp:~cando/activity-log-manager/925638 into lp:activity-log-manager.

References

[Merge] lp:~cando/activity-log-manager/925638 into lp:activity-log-manager
From: Stefano Candori, 2012-02-24