----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://mattr.info/r/163/#review155 -----------------------------------------------------------
All the extra whitespace (highlighted in red) need to be removed from the patch. I'm still not sure this is the right way. The reason I say is because we introduce two different behaviors depending upon how many contacts a metacontact has in it. Anyways, submit another revision for the whitespace and perhaps we can get a discussion on kopete-devel started about this. - Matt On 2008-02-18 21:23:04, Adam Russell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://mattr.info/r/163/ > ----------------------------------------------------------- > > (Updated 2008-02-18 21:23:04) > > > Review request for Kopete. > > > Summary > ------- > > Fixes two Oscar synchronization issues: > 1. Moving a contact from one group to another on the server now moves the > contact on the client. > 2. Once a contact is removed from the server, an option is now given to > delete the metacontact from the client. > > Also added a TODO for group synchronization. > > > Diffs > ----- > > trunk/KDE/kdenetwork/kopete/protocols/oscar/oscaraccount.cpp > trunk/KDE/kdenetwork/kopete/protocols/oscar/oscarlistnonservercontacts.cpp > > Diff: http://mattr.info/r/163/diff > > > Testing > ------- > > 1. Manipulated SSI via another client so that the group for a contact does > not match Kopete's group. Signed on and confirmed group synchronization. > 2. Removed a contact via another client. Signed on, received modified > dialog, and confirmed metacontact removal after selecting the deletion option. > > > Thanks, > > Adam > > _______________________________________________ kopete-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kopete-devel
