Review: Needs Fixing
First of all, thank you for working on this!
As commented on IRC, I don't really see a point for storage_state, num_events
and result_type in the event deletion method. The original proposal just had
time_range and event_templates.
(I'm also unconvinced that this methods performance is critical enough to
warrant the complexity of duplicating the query. I think I may prefer a
FindEventIds+DeleteEvents implementation, like Seif proposed.)
Your team Zeitgeist Framework Team is subscribed to branch lp:zeitgeist.
Mailing list: https://launchpad.net/~zeitgeist
Post to : email@example.com
Unsubscribe : https://launchpad.net/~zeitgeist
More help : https://help.launchpad.net/ListHelp