-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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(nexmo.js): add support for Nexmo/Vonage SMS #136
Conversation
providers/nexmo/README.md
Outdated
@@ -0,0 +1,15 @@ | |||
# Notifire Nexmo Provider | |||
|
|||
A Nexmo sns provider library for [@notifire/core](https://github.com/notifirehq/notifire) |
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.
Actually sns
seems like the boilerplate bug @scopsy
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.
@ranrib on it, thanks!
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.
Looks amazing! Know a few folk who will be really excited for this one 🎉
If you could just run prettier on the provider file, until we will fix the precommit
hook to run it automatically
options.content, | ||
{}, | ||
(err, responseData) => { | ||
if (err) reject(err); |
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.
if (err) reject(err); | |
if (err) return reject(err); |
Missing a return here :)
from: string; | ||
} | ||
) { | ||
this.vonageClient = new Vonage({apiKey: config.apiKey, apiSecret: config.apiSecret}); |
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.
Could you please run prettier in this file? 💅 @matthiez is taking a look at why it's not working on the precommit hook
providers/nexmo/README.md
Outdated
@@ -0,0 +1,15 @@ | |||
# Notifire Nexmo Provider | |||
|
|||
A Nexmo sns provider library for [@notifire/core](https://github.com/notifirehq/notifire) |
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.
@ranrib on it, thanks!
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/dimagrossman/docs/GMLvgrzKCPPdkVt1bHnFwYja2kgv |
All fixed @scopsy |
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Supports Vonage/Nexmo SMS
What is the current behavior? (You can also link to an open issue here)
implements Add new Nexmo provider #26
What is the new behavior (if this is a feature change)?
Other information: