Skip to content
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

[NEW] Enable twilio whatsapp template #1391

Merged
merged 14 commits into from
Sep 6, 2023
Merged

[NEW] Enable twilio whatsapp template #1391

merged 14 commits into from
Sep 6, 2023

Conversation

bhardwajaditya
Copy link

@bhardwajaditya bhardwajaditya commented Aug 2, 2023

Proposed changes (including videos or screenshots)

Closes: #1385

Issue(s)

Steps to test or reproduce

Further comments

@bhardwajaditya bhardwajaditya requested a review from ear-dev August 2, 2023 08:42
@ear-dev
Copy link

ear-dev commented Aug 31, 2023

@bhardwajaditya please add logging that shows exactly what is in the call we make to twilio.

  • Include all the parameters...... twilio team has suggested that they are only getting the to and from fields.
  • Since we use the twilio library method client.messages.create() it could be possible that the library is not accepting our extra params. Is there a logging function upon POST that we can use to see exactly what it is sending.

@ear-dev
Copy link

ear-dev commented Aug 31, 2023

@bhardwajaditya Looking again at the twilio docs it seems that they are showing a way to log what the twilio library is doing by using the .then() part?

client.messages
.create({
contentSid: 'HX...XXX',
from: 'MG...XXX',
contentVariables: JSON.stringify({
1: 'Name'
}),
to: 'whatsapp:+18551234567'
})
.then(message => console.log(message.sid));

FYI - in regard to the twilio nodejs library, the rep sent me this link for modifications: https://github.com/twilio/twilio-node/blob/main/advanced-examples/custom-http-client.md

@ear-dev ear-dev merged commit df180d2 into develop Sep 6, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WhatsApp interactive messages
3 participants