[Bug 32267] Implement handling Client.Observer.DelayApprovers

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon Jan 3 13:52:59 CET 2011


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

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.

More information about the telepathy-bugs mailing list