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

feat(provider): send notifications/messages to whatsapp or as text SMS using Twilio #91

Merged
merged 8 commits into from
Mar 10, 2023

Conversation

sadarunnisa-sf
Copy link
Contributor

@sadarunnisa-sf sadarunnisa-sf commented Sep 22, 2022

send notifications/messages to whatsapp or as text SMS using Twilio

GH-88

Description

Need to send SMS or whatsapp messages via Twilio. (https://www.twilio.com/docs/sms, https://www.twilio.com/docs/whatsapp) - Twilio is a modern communication API Used by developers for establishing communications. Twilio can be used as provider for sending SMS or Whatapp notifications

Fixes #88
This extension uses AWS SNS as SMS provider. SNS is not a good option for SMS as its delivery is unreliable. Please refer #88

Type of change

FEAT - Added Provider

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Test A
    Unit test for twilio provider. Created twilio account and used sandbox to send SMS and whatsapp messages. Created sandbox whatsapp channel for whatsapp messages. Created messaging service SID to send SMS using twilio provider. We get "accountSid", "authToken" as from twilio for integration. Set required configurations in unit tests for the same and run test to send sms, whatsapp message and whatsapp message with media. (src/tests/unit/twilio.provider.unit.ts)

Checklist:

  • Performed a self-review of my own code
  • npm test passes on your machine
  • New tests added or existing tests modified to cover all changes
  • Code conforms with the style guide
  • API Documentation in code was updated
  • Any dependent changes have been merged and published in downstream modules

@sonarcloud
Copy link

sonarcloud bot commented Sep 23, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@yeshamavani
Copy link
Contributor

@sadarunnisa-sf please rebase the branch

@yeshamavani yeshamavani changed the title feat(provider): send notifications/messages to whatsapp or as text SM… feat(provider): send notifications/messages to whatsapp or as text SMS using Twilio Mar 2, 2023
@sonarcloud
Copy link

sonarcloud bot commented Mar 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@yeshamavani yeshamavani merged commit f9d9799 into sourcefuse:master Mar 10, 2023
@yeshamavani
Copy link
Contributor

unable to resolve the code smell

@sfdevops
Copy link
Contributor

🎉 This PR is included in version 5.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Need twilio support for SMS/Whatsapp
4 participants