Le mercredi 04 juin 2008 à 17:08 +0200, Morgan Collett a écrit :
> The main code to review is at:
> http://dev.laptop.org/git?p=users/morgan/sugar;a=shortlog;h=6298 (3
> most recent patches).

As bundle_id is passed to both constructor, you could move it to
BaseInvite.__init__


Didn't read code carefully but InviteButton and InvitePalette still
contain lot of:

if shared:
  ...
else:
  # private


Maybe it would be worth to abstract these 2 classes too if possible?



        G.

_______________________________________________
Sugar mailing list
Sugar@lists.laptop.org
http://lists.laptop.org/listinfo/sugar

Reply via email to