-
Notifications
You must be signed in to change notification settings - Fork 55
RFE: remove pusher.com dependency #50
base: master
Are you sure you want to change the base?
Conversation
1c61add
to
efa61be
Compare
👋 Hi @nonameentername: I just wanted to drop a note to acknowledge this pull request and say thanks for taking this on. Based on the discussion in #35, it seems like several people could be happy to try out this pull request. 😄 We're heads down on a handful of other priorities at the moment, so it's unlikely that we'll be able to give this pull request the thorough review it deserves in the near term. However, if people are able to make use of these changes by checking out your branch, then I think that's awesome. ✌️ |
ece53fc
to
d19d22e
Compare
d19d22e
to
7cf678c
Compare
What's the status on this???? This looks awesome. |
Thanks! I'll resolve conflicts so it can be merged. |
Dear @jasonrudolph , it is a bit over a year since the pull request was proposed. I hope that you have some time in the near future to consider it for inclusion. From a data protection compliance perspective, pusher is yet another data processor, which introduces again more work on contracts and data protection policies. Also here, an on-prem solution would allow for an easier adoption of teletype. Best, |
The |
I've forked this PR and performed a merge with the latest develop. However, I don't have the means to test it at the moment. |
RFE: remove pusher.com dependency - Merged
@nonameentername merged my changes so this PR should be up to date. |
Implemented #35