[Bug 28197] Account.I.External: support for immutable accounts in Mission Control

[bugzilla-daemon at freedesktop.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Keywords|patch                       |
         AssignedTo|telepathy-bugs at lists.freede |danielle.madeley at collabora.
                   |sktop.org                   |co.uk

--- Comment #1 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2010-05-21 02:59:28 PDT ---
Reviewing Danielle's branch from the perspective of the spec, as if it had been
a tp-spec branch:

If this is being worked on actively, it should get into the spec as a draft.

The <interface name=""> should end with .DRAFT for draft API.

It's not instantly clear whether this is an interface that will exist (in the
Interfaces property) on all accounts (so its semantics are really more like
"PotentiallyExternal"), or only on accounts that are in fact External. Either
way, the introductory docstring should explain.

I actually think that since MC core changes are needed for this interface
*anyway*, it should probably be omitted from Interfaces if External would be
FALSE; this means the External property is unnecessary. (It'll still need to
appear in the keyfile.)

Before undrafting this, we need a usable implementation in MC (which is Bug
#28192).

+    <property name="ExternalName" tp:name-for-bindings="External_Name"
+      type="s" access="read">
+      <tp:docstring>
+        A human-readable name for program/interface used to edit the
+        account.

Is this localizable (how?), or is it "in the C locale" like
Protocol.EnglishName in my branch for Bug #20774? (I'd suggest the latter, in
which case please consider stealing the wording from that branch.)

+        A D-Bus well-known name that can be activated and requested to show
+        the appropriate account settings page over D-Bus.
+
+        FIXME: specify the object-path and interface for account editing.
+
+        <tp:rationale>
+          Using

Needs more <p>.

Is the intention that activating the process with
org.freedesktop.DBus.Peer.Ping is sufficient, or are you going to define an
object path and method that should be called? Alternatively, you could have
some text like this:

    No semantics are currently defined for the object paths and interfaces
    implemented by processes with this well-known name: it acts as an
    opaque identifier for the interface used to edit the account.

-- 
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.