-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Send buttons and Lists #756
Conversation
Draft to add list and buttons. Thanks to @purpshell
Button example: //text button
let button = new Buttons("body",[{body:"bt1"},{body:"bt2"},{body:"bt3"}],"title","footer");
client.sendMessage(message['from'], button);
//media button ( Testing, images works fine )
let media = MessageMedia.fromFilePath("image.png");
let button = new Buttons(media,[{body:"bt1"},{body:"bt2"},{body:"bt3"}],"title","footer");
client.sendMessage(message['from'], button,{'caption':"body"}); Button tests needed : |
I received this, even updating the project |
have you cloned the PR????? are you using NPM? |
i use npm |
List example: let sections = [{title:'sectionTitle',rows:[{id:'customId', title:'ListItem1', description: 'desc'},{title:'ListItem2'}]}];
let list = new List('aaa','btnText',sections,'Title','footer');
client.sendMessage(message['from'], list); Lists aren't working on business right now Tests needed: |
Tested on Android Business, and iOS Standard, and its working. |
I've been testing this DRAFT, and took notes of some things. Number 1: Number 2: |
Makes more sense since buttons_message is way different than templateMessage which msg.buttons is trying to act with |
Thanks for your help! |
Ignore number 1, as it is already solved by #386 |
Any idea when we can have this shipped? |
@pedroslopez Please add this functionality! |
It seems that the most recent WhatsApp version (2.2134.9) broke the sending of this buttons: |
Hmmm it's working here, are you using whatsapp-web.js@1.14.1 ? |
Yep, I'm using the latest whatsapp-web.js@1.41.1. Before the WhatsApp update it was working perfectly, This error seems to be an internal "error" from WhatsApp not necessarily from the whatsapp-web.js library. |
Not Working on MD Version @purpshell |
|
I tested it for two weeks and I don't have any problem. |
Problem only for Beta MultiDevices |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I spent some time testing some things to see if I could come up with a solution, but I wasn't able to come up with this excellent solution
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Merge time |
@Goomer2 , could you send us some code? |
I'm using biz.
The error is on Whatsapp Web. |
|
Maybe you could look into how these guys are doing it |
Maybe you could look into how these guys handled it: |
This reverts commit b60029e.
Add list and buttons.
Thanks to @purpshell