[Telepathy] Review of MissionControl spec

[Simon McVittie]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Tue, 22 Jan 2008 at 19:19:21 +0100, Xavier Claessens wrote:
> On lun, 2008-01-21 at 21:43 +0000, Simon McVittie wrote: 
> > Documented as returning (s, o) "if it's online". What happens if it's
> > offline needs documenting too.
> 
> Ok so it should return the object path or "" if offline?

Can't do that. Object paths aren't nullable. You could use a hack like
returning "/", but raising an appropriate error would be better
(NotAvailable springs to mind as appropriate).

> We could set supress_handler=false like that MC gets NewChannel but MC
> will think it's an incoming channel even if it should be outgoing.

Dangerous. We don't want to conflate incoming/outgoing,
handled/unhandled and whatever other boolean distinctions we're making.

What do you mean by "incoming"/"outgoing" and is it the same thing Mardy
means?

> In fact Empathy updates each value separately directly when you write
> it, so there is not a "Cancel" and "Ok" buttons but only "Close" as
> everything already got applied. I did that because GNOME applications
> generally prefer instant-apply and NMC API makes that easy. However with
> "new" behaviour of NMC is to reconnect the account each time a param is
> changed so I really should change that in Empathy UI.
> 
> Not sure what's the best...

NMC shouldn't do that. I'm not sure what it should do, but not that.

The Nokia UI doesn't let you change account params unless the account is
disabled, which seems somewhat reasonable - GNOME-style insta-apply isn't
really feasible when these params don't take effect until the next time you
reconnect!

> > RegisterChandler
> > No. Wrong interface. Try again.
> 
> You mean that should be moved to another interface or that shouldn't
> exist at all?

I mean it should be on a ChannelDispatcher interface (or something),
that might not even be part of the same service, and most importantly, that
we don't need to talk about yet in order to get a good AccountManager
design :-)

    Simon
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFHlj0yWSc8zVUw7HYRAqNIAKDqnvSxn1XwJHY6Xl70YyjUUHtz1QCg1AHe
cSz6cGo3WCxa8QBap7Wb6Gs=
=rFyD
-----END PGP SIGNATURE-----