I updated telepathy-spec-mc branch [1] and the html spec [2]. I think all problems mentionned here are more or less fixed now.<br><br>Remaining things:<br><br>1) I should add {Get|Set}IconName{Updated} to the Account interface?
<br>2) If no presets are provided, who decides DisplayName/IconName?<br>3) ConnectionManager interface needs Capability management [3].<br>4) In Chandler file spec, one .chandler file can handle many channel types, but there is only one TypeSpecificCapabilities field. We need one capability field per channel type, or only allow one channel type per file.
<br>5) We should set possible errors for each calls<br>6) Write more comprenhensible doc with better English...<br>7) Reviewing!<br><br>Xavier Claessens.<br><br>[1] <a href="http://projects.collabora.co.uk/~monkey/telepathy-spec-mc/">
http://projects.collabora.co.uk/~monkey/telepathy-spec-mc/</a><br>[2] <a href="http://projects.collabora.co.uk/~xclaesse/spec.html">http://projects.collabora.co.uk/~xclaesse/spec.html</a><br>[3] <a href="http://projects.collabora.co.uk/~monkey/telepathy-spec-rob-roundup/">
http://projects.collabora.co.uk/~monkey/telepathy-spec-rob-roundup/</a><br><br><div><span class="gmail_quote">2007/10/21, Xavier Claessens <<a href="mailto:xclaesse@gmail.com">xclaesse@gmail.com</a>>:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi,<br><br>I opened a darcs branch to add MissionControl's spec into<br>telepathy-spec[1]. You can see generated html [2] I'm not sure it's the<br>right place but if I make a separate package I have to duplicate all
<br>generator code from tp-spec... Also, will we add MC interfaces to<br>tp-glib? Maybe in a libtelepathy-glib-mc.so? Like that we can use the<br>same code generator to implement MC.<br><br>Problems I see:<br><br>1) org.freedesktop.Telepathy.MissionControl.ProtocolManager
: Is that<br>interface useful at all? libtelepathy already has code to do such<br>things. I think we should think about a good profile system instead.<br>Information in NMC profiles:<br><br> Manager=gabble<br> Protocol=jabber
<br> DisplayName=Google Talk<br> IconName = empathy-proto-google-talk<br> ConfigurationUI = jabber<br> Capabilities = chat-p2p, chat-room, chat-room-list, voice-p2p,<br> split-account, supports-avatars, supports-alias
<br> SupportedPresences = offline,available,away,extended-away,<br> do-not-disturb<br> VCardField = X-Jabber<br> DefaultAccountDomain = <a href="http://gmail.com">gmail.com</a><br> Default-$param = $value
<br><br> - DisplayName: Should be internationalized.<br> - IconName: Depends on who install the icon PNGs<br> - ConfigurationUI: In N800 that's a string with the path to the UI<br>plugin to configure parameters of the account, that totally depends on
<br>the client. For Empathy it's just a string to choose between available<br>hard-coded UIs.<br> - Default-$param = $value: That depends on client's policy, for example<br>should we ignore ssl errors to connect gmail accounts? It could also be
<br>set by SIP providers to configure everything for their service.<br><br>So who should install profiles? If each client install his set of<br>profiles and the user install kopette and Empathy, he will have 2<br>profiles for gabble... If the CM install profile how could we have
<br>translated fields, icons, etc? I don't know how to do that right.<br><br>2) ListAccounts, FindAccounts and GetOnlineAccounts does the same thing:<br>finding accounts corresponding to some criterias. ListAccounts can be
<br>done using FindAccounts(NULL) and GetOnlineAccounts can be done using<br>FindAccounts(["status", connected]).<br><br>3) AccountStatusChanged is on AccountManager and GetConnectionStatus is<br>on Account... They should be both on the same object, but which one?
<br><br>4) ConnectAll: is that useful? The default presence is not defined in<br>the spec... I think it can be easily replaced by SetGlobalPresence. Or<br>maybe we should define in the spec that MC must save presence for all
<br>accounts and calling ConnectAll will restore saved presence? In that<br>case ConnectAll could be renamed to ResorePresence?<br><br>5) RequestChannel: description says "putting the account online if<br>necessary". Ok but with which presence? I think that method should just
<br>return an error NOT_CONNECTED in that case.<br><br>6) AccountStatusChanged: It gives the account's presence, is that<br>useful? We have a signal for presence update on the account.<br><br>7) RequestChannel: handle_type should be u.
<br><br>That's all for now, I will update my branch with your suggestions.<br><br>Xavier Claessens.<br><br>[1] <a href="http://projects.collabora.co.uk/~monkey/telepathy-spec-mc/">http://projects.collabora.co.uk/~monkey/telepathy-spec-mc/
</a><br>[2] <a href="http://projects.collabora.co.uk/~xclaesse/spec.html">http://projects.collabora.co.uk/~xclaesse/spec.html</a><br><br></blockquote></div><br>