Received-SPF: pass (google.com: best guess record for domain of linux-bluetooth-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <1583398524-18749-1-git-send-email-ajay.kishore@intel.com>
 <1583398524-18749-6-git-send-email-ajay.kishore@intel.com> <CABBYNZKUnVWpZxBVW12JnoY1ZV+_jwsO90+M8-kJNYjaA0kBDw@mail.gmail.com>
In-Reply-To: <CABBYNZKUnVWpZxBVW12JnoY1ZV+_jwsO90+M8-kJNYjaA0kBDw@mail.gmail.com>
From:   Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date:   Thu, 5 Mar 2020 11:58:47 -0800
Message-ID: <CABBYNZJmONdE5-0mczy+RX3NwVKbM0evqChBzuCi97W+3UxthA@mail.gmail.com>
Subject: Re: [PATCH v3 6/6] doc/obex-api: Update documentation
To:     Ajay Kishore <ajay.kishore@intel.com>
Cc:     "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

Hi Ajay,

On Thu, Mar 5, 2020 at 11:53 AM Luiz Augusto von Dentz
<luiz.dentz@gmail.com> wrote:
>
> Hi Ajay,
>
> On Thu, Mar 5, 2020 at 1:21 AM Ajay Kishore <ajay.kishore@intel.com> wrote:
> >
> > This adds documentation with the conversation listing feature
> > ---
> >  doc/obex-api.txt | 50 ++++++++++++++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 50 insertions(+)
> >
> > diff --git a/doc/obex-api.txt b/doc/obex-api.txt
> > index f39355a..ef81460 100644
> > --- a/doc/obex-api.txt
> > +++ b/doc/obex-api.txt
> > @@ -712,6 +712,44 @@ Methods            void SetFolder(string name)
> >                         Possible errors: org.bluez.obex.Error.InvalidArguments
> >                                          org.bluez.obex.Error.Failed
> >
> > +               array{object, dict} ListConversations(string folder, dict filter)
>
> If we are going to return Message1 objects this should reuse
> ListMessages and then use the Type filter to specify the types of
> message to list, if that is not present then it should list
> everything.

Actually let me take this back, the properties appear to be very
different so combining those objects might not be reusing much, so
lets do this using ListConversation and Conversation1 objects with
their own properties, that said the names I suggested bellow still
hold.

> > +                       Returns an array containing the conversations found in the
> > +                       given subfolder of the current folder, or in the current
> > +                       folder if folder is empty.
> > +
> > +                       Possible Filters: MaxListCount, LastActivityBegin, LastActivityEnd,
> > +                       ReadStatus, Recipient
> > +
> > +                       Each message is represented by an object path followed
> > +                       by a dictionary of the properties.
> > +
> > +                       Properties:
> > +
> > +                               string id:
>
> Properties should use the same format as methods in D-Bus, ID might be better.
>
> > +
> > +                                       Conversation unique identification
> > +
> > +                               string last_activity:
>
> LastActivity
>
> > +                                       Conversation timestamp for the last activity
> > +
> > +                               string chat_state:
> > +
> > +                                       Conversation current chat state of the participants
>
> ChatState
>
> > +                               string presence_availability:
> > +
> > +                                       Conversation  participants availability
>
> Presence
>
> > +                               string presence_text:
>
> We could reuse Subject here.
>
> > +                                       User defined status of the conversation
> > +
> > +                       Possible errors: org.bluez.obex.Error.InvalidArguments
> > +                                        org.bluez.obex.Error.Failed
> > +
> > +
>
> I though we discussed about this but it seems that the properties here
> have no relation to Message1 which appears to be the objects you are
> creating so you will have to introduce these properties also in
> Message1, besides we need to include a new possible value for Type
> e.g. "Conversation"
>
> >                 void UpdateInbox(void)
> >
> >                         Request remote to update its inbox.
> > @@ -799,6 +837,18 @@ Filter:            uint16 Offset:
> >                         Possible values: True for high priority or False for
> >                         non-high priority
> >
> > +               string LastActivityBegin:
> > +
> > +                       Filter conversations by starting period.
> > +
> > +                       Possible values: Date in "YYYYMMDDTHHMMSS" format.
> > +
> > +               string LastActivityEnd:
> > +
> > +                       Filter conversations by ending period.
> > +
> > +                       Possible values: Date in "YYYYMMDDTHHMMSS" format.
> > +
> >  Message hierarchy
> >  =================
> >
> > --
> > 2.7.4
> >
>
>
> --
> Luiz Augusto von Dentz


-- 
Luiz Augusto von Dentz