Something looks to be ascrew if an event has multiple subject? But it looks 
like that was also the case before this branch..? Do we have a test for this?

Can you re-insert the comment about returning None for missing events - I think 
it's nice to note there that it is our API contract.

But looks good to me otherwise
-- 
https://code.launchpad.net/~seif/zeitgeist/clean-up-get-events/+merge/40529
Your team Zeitgeist Framework Team is requested to review the proposed merge of 
lp:~seif/zeitgeist/clean-up-get-events into lp:zeitgeist.

_______________________________________________
Mailing list: https://launchpad.net/~zeitgeist
Post to     : zeitgeist@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zeitgeist
More help   : https://help.launchpad.net/ListHelp

Reply via email to