match_jingle_action should be curried: almost every use is lambda e: jp.match_jingle_action(e.query, 'session-initiate') or similar. Extract NewChannels-checking boilerplate. Rename make_channel_proxy: wrap_channel should be used for channels, and the former only for SessionHandlers and StreamHandlers.
Done the second one in a branch; will rearrange patches and put it up for review at some point.
I never did finish extracting the NewChannels-checking boilerplate, but I did add a curried and flipped version of match_jingle_action, namely action_predicate.
-- GitLab Migration Automatic Message -- This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/telepathy/telepathy-gabble/issues/29.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.