[feedhenry-dev] [Feedhenry-raincatcher] Ability to subscribe to more than one event at a time

Wojciech Trocki wtrocki at redhat.com
Mon Oct 23 09:02:12 UTC 2017

Hi Michael

There are two side effects in Feedhenry sync that are important for your

1) Line that was forwarded to us is responsible for triggering notification.
If you have specific notification then global notification will not be


IMHO both should be triggered as this is how most of the event
implementation works.
This is trivial fix, but it may cause some backwards compatibility problems.

2) When you register notifications for specific datasets previous notifier
will be removed

IMHO this may relay on something like event emitter, but that will require
a lots of changes in the library.

*With that in mind - we cannot have more than 1 notification handler per
RainCatcher team can provide wrapper for that method in RainCatcher that
will allow us to notify more than one listener per dataset.

Sending to feedhenry-dev for other opinions, before executing any actions.

On Fri, Oct 20, 2017 at 6:07 PM, Michael Hearne <mhearne at redhat.com> wrote:

> There is a bug in sync where we cannot subscribe to events more than once
> at a time. The following fails:
> workorderService.subscribeToDatasetUpdates(() => {
>     ...
>   });
> https://github.com/feedhenry/fh-sync-js/blob/6392b02813b09e2
> 52b1e7537249ce50dd8f5e5cb/src/sync-client.js#L360-L361
> --
> mhearne at redhat.com
> <https://red.ht/sig>
> @m <https://twitter.com/mickhearne>ickhearne  |  IM: #mhearne
> _______________________________________________
> Feedhenry-raincatcher mailing list
> Feedhenry-raincatcher at redhat.com
> https://www.redhat.com/mailman/listinfo/feedhenry-raincatcher



Red Hat Mobile <https://www.redhat.com/>

IM: wtrocki
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/feedhenry-dev/attachments/20171023/ea5a15cd/attachment.htm>

More information about the feedhenry-dev mailing list