[Bug 32267] Implement handling Client.Observer.DelayApprovers

[bugzilla-daemon at freedesktop.org]

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

Simon McVittie <simon.mcvittie at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
  Status Whiteboard|                            |review-, more test coverage
                   |                            |please
         AssignedTo|telepathy-bugs at lists.freede |jonny.lamb at collabora.co.uk
                   |sktop.org                   |

--- Comment #2 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2011-01-03 04:52:59 PST ---
I can't find a bug# regarding the spec, but it looks good.

Implementation looks good, but I'd also like to see a test in which the
Observer follows the pattern from Bug #27860 (with either Claim or HandleWith),
and as a result, ADO isn't called *at all*; for the cases mentioned in that bug
(and in the spec branch here), it seems as though that's the important thing we
want.

Relatedly, it would be really great (but isn't a merge blocker for this) if you
could fix Bug #27860 (document that observers can act like non-interactive
approvers). Perhaps the situations Will mentions there could help to make this
bug's spec rationale more concrete, too.

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