[Code-review] review: Gadget current-activity branch
Dafydd Harries
dafydd.harries at collabora.co.uk
Thu Aug 14 10:11:51 EDT 2008
Ar 12/08/2008 am 12:06, ysgrifennodd Guillaume Desmottes:
> Le lundi 11 août 2008 à 17:13 +0100, Dafydd Harries a écrit :
> > >+ # (room, activity id)
> > >+ self.current_activity = ('', '')
> >
> > Why not make this an activity object? (Or None for no current activity.)
>
> Then that implies that if Gadget is notified about a current-act change
> involving an activity which is not in Gadget's model, we'll have to set
> buddy's current-act to None.
> Is that something we want? IIRC, we agreed that Gadget should always fw
> notifications and that's client's job to ignore this notification if
> that's an unknown activity.
Well, we could still use an Activity object: it doesn't have to be in the
model.
> > >+ # FIXME: what about buddies who are in activity views as
> > >+ # participants ?
> >
> > I don't understand this comment. Could you elaborate?
>
> I rephrased it to:
> # FIXME: We only notify changes to buddy views. But buddies can also
> # be in activity views (as participants). We should send notification
> # messages in that case too.
Much clearer.
--
Dafydd
More information about the Code-review
mailing list