This message was deleted.
# helpdesk
s
This message was deleted.
d
yeah that makes sense to filter out the events that you'd care about.. are you thinking having multiple webhooks, with each receiving a subset of events?
f
I only have a single webhook and in my case, a single filter list for all the hooks will work like a charm. But at the time of writing, I thought why not generalize it 🙂. I'm also interested in adding this and sending a PR if that's ok.
d
sure, that'd be great.. as long as we can do it in a backwards compatible fashion to not break existing users
f
Great! I'll keep that in mind.
🙌 1
@dry-elephant-14928 Hi. I made a PR#547 that adds
FilteredNotifier
. I think I might have made it a bit complex but it made testing easier and all of the logic was reused.