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 : [email protected] Unsubscribe : https://launchpad.net/~zeitgeist More help : https://help.launchpad.net/ListHelp
