Hi Denis

I would really like that we get a public discussion about such addition

18392 Announcer should allow to subscribe on specific instances of events
        https://pharo.fogbugz.com/f/cases/18392

AnnouncementDeliveryCondition introduced. It can be used in place of Announcement class in all subscription methods:

announcer when: (ValueChanged where: [:change | change oldValue = #expected]) do: [:ann | ]

So since there was no real discussion I check the class comments:

Good class comment! Thanks.

Now I still would like to understand: AnnouncementDeliveryCondition is a wrapper on a Announcement? I imagine yes.

Probably we should add that in the class comment.

Now I do not get why this filtering should be in the delivery because the risk is that we will get even more announcements in announcements queue. Of course it will probably reduce the number of announcement
emitted. So I would like to know more.

Stef

Reply via email to