added PLIVO as a SMS service provider

Abhishek-plivo commented 7 years ago

Hey, this is Abhishek from Plivo . We saw this cool npm package and thought of adding Plivo as a SMS provider in this module . Please find the commit details mentioned below as well review the commits :-

src/providers/sms/plivo.js : added SmsPlivoProvider module for sending SMS via PLIVO
src/models/provider-sms.js : added plivo in sms-provider model
src/providers/sms/index.js : imported SmsPlivoProvider and enabled it to be used by giving plivo as the provider name
README.md : added documentation of initializing sdk with plivo.
.gitignore ignored automatically generated macbook files (.DS_Store)

BDav24 commented 7 years ago

Hi @Abhishek-plivo thanks for your pull request! We replaced request dependency by node-fetch, but don't worry I'll update your code.

BDav24 commented 7 years ago

https://github.com/notifme/notifme-sdk/commit/67ea41be90afebf8443b685971bda06764c6c082

Abhishek-plivo commented 7 years ago

Thanks, @BDav24 . I see Travis CI build failed . Does it need to change something in our commits ?

BDav24 commented 7 years ago

Everything's okay: https://travis-ci.org/notifme/notifme-sdk/builds/265611020 (I forgot to tag my commit with the reference of this PR, that's why it doesn't appear here)

Abhishek-plivo commented 7 years ago

cool , thanks

notifme / notifme-sdk

added PLIVO as a SMS service provider #10