[Bug 28200] TpBaseContactList

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=28200

--- Comment #15 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2010-06-04 09:10:38 PDT ---
(In reply to comment #13)
> I've implemented this in smcv/contact-list-manager. Thinking about it, I don't
> think the contact's state should visibly change when pre-approving like this,
> so I didn't implement that bit.

... but that's OK, because the spec didn't say that I should either :-)

I've rebased smcv/contact-list-new-signals onto the new API. The only
non-automatic changes were to resolve a conflict in this commit:

http://git.collabora.co.uk/?p=user/smcv/telepathy-glib-smcv.git;a=commitdiff;h=676a5d18a9c2b351649df76dd929336ad9714302

and to assert about ContactsChanged signals in
/contact-lists/add-to-publish/pre-approve in this commit:

http://git.collabora.co.uk/?p=user/smcv/telepathy-glib-smcv.git;a=commitdiff;h=1402e88674e70177de4d2ae5432773bbe2026d4b

May I consider smcv/contact-list-new-signals to have been reviewed, conditional
on getting the spec merged? If so, I'll use that as my new baseline for
changes.

wip-contact-list-new-methods still needs rebasing, which will be easy but
time-consuming; then I can port Gabble to this API, and fix errata in the spec.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.