Re: Bookmark datatype
- From: John Carr <john carr unrouted co uk>
- To: Renato Araujo <renatox gmail com>
- Cc: conduit-list gnome org
- Subject: Re: Bookmark datatype
- Date: Wed, 12 Sep 2007 20:28:46 +0100
Excellent!
On Wed, 2007-09-12 at 16:19 -0300, Renato Araujo wrote:
> Thanks john,
>
> I created the set/get data method, and already have a delicious module
> working with 2 delicious accounts, but this need more tests maybe I
> finish on this weekend, I will move the name to Bookmark and will send
> a complete patch to all module.
>
> I'm thinking about create a dataprovider to mozilla bookmark, then I
> can test this together.
>
> BR
> Renato
>
> On 9/12/07, John Carr < john carr unrouted co uk> wrote:
> Hi
>
> Sorry for not replying sooner. The datatype looks good, an
> excellent
> first contribution. Before I commit this, could you have a
> think (and
> feedback to us) about:
>
> I think there is a problem in compare(). It really needs to
> decide which
> is the newest version of the data, but i'm not sure how you do
> that in
> the bookmark case..
>
> You don't need the get / set bookmark string, but it makes
> things a lot
> easier for the poor soul who is implementing converters ( e.g.
> bookmark
> to/from file for syncing to a file server). If you could get
> the setter
> working that would really kick ass.
>
> Its a picky thing to ask, put i'd prefer to call it
> "Bookmark", rather
> than "BookmarkEntry".
>
> Thanks for your work on this!
>
> Did
>
> John
>
> On Fri, 2007-09-07 at 20:12 -0300, Renato Araujo wrote:
> > Hi all,
> > I am creating a new dataprovider to delicious sync, but for
> this i
> > will need a new datatype to store the bookmark info. What
> you think
> > about this?
> > Attached on this message, is the code of my new datatype.
> >
> > BR
> > Renato Filho
> > _______________________________________________
> > Conduit-list mailing list
> > Conduit-list gnome org
> > http://mail.gnome.org/mailman/listinfo/conduit-list
>
>
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]