Re: Bookmark datatype

[John Carr <john carr unrouted co uk>]

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