-
-
Notifications
You must be signed in to change notification settings - Fork 413
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WhatsApp Support #808
Comments
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 |
Hi, |
Hi @caronc , Cheers 😀 |
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 |
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 But if there is no persistent requirements for WhatsApp, I'll take another look at the efforts |
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. |
Merged |
📢 What is the name of the service?
Whatsapp
🔗 Provide some details about the service that can help with it's development.
The text was updated successfully, but these errors were encountered: