caronc / apprise

Apprise - Push Notifications that work with just about every platform!
https://hub.docker.com/r/caronc/apprise
BSD 2-Clause "Simplified" License
11.34k stars 401 forks source link

WhatsApp Support #808

Closed t0mer closed 1 year ago

t0mer commented 1 year ago

:loudspeaker: What is the name of the service? Whatsapp

:link: Provide some details about the service that can help with it's development.

caronc commented 1 year ago

The only catch with WhatsApp is that once you establish a connection to the server (via QRCode), it needs to persist somewhere so that the connection can be re-used.

This requires the #749 issue to be completed

t0mer commented 1 year ago

Hi, Ma-nish is working with the official whatsapp cloud api and does not require qrcode. When you open the api you get a permanent token and thats it 🙃

t0mer commented 1 year ago

Hi @caronc , Any news?

Cheers 😀

caronc commented 1 year ago

The request can't be supported yet because there is no persist storage with Apprise. After you authenticate with WhatsApp, you need to hang on to the credentials to make the notification, plus additional ones thereafter

caronc commented 1 year ago

The only other comment I'd make is I will likely avoid using the MaNish library as Apprise only needs to leverage a single POST to the WhatsApp API to send the notification. Possibly with attachment support it will post to 2-3 more URLs. The goal is to keep the requirements.txt as required only. Consider it the way the file is now and supporting 90 some endpoints.

But if there is no persistent requirements for WhatsApp, I'll take another look at the efforts

caronc commented 1 year ago

~Still a ways to go, but i believe i have a working version right now.~ I can't seem to tell if the direct messages are working (outside of the templates which are). I'm not sure if that's because i'm not paying for my account at this time or not.

Edit: Test Coverage at 100% now; it's back in all of your hand just to give me the thumbs up or down on if this is complete enough.

I would really be interested in anyone's feedback; instructions to test this out are in side the merge request.

caronc commented 1 year ago

Merged