-
Notifications
You must be signed in to change notification settings - Fork 89
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
add Ghasedak Sms Provider #101
add Ghasedak Sms Provider #101
Conversation
provider/ghasedak/ghasedak.go
Outdated
// Retrieving required data from 'ghasedak' sections of config.yaml | ||
type Config struct { | ||
APIToken string `yaml:"api_token"` | ||
PhoneNumbers []string `yaml:"phone_numbers"` |
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.
PhoneNumbers is not used!
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.
they normally come from the message
obj
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.
'PhoneNumbers' is sender phone number
I will delete this config because not usable in Ghasedak API
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.
We call this "originator" and it's also on the Message https://github.com/messagebird/sachet/blob/master/sachet.go#L9
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.
in last pull request, I delete config
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.
in Kavenegar Provider : Config PhoneNumbers Exist
https://github.com/messagebird/sachet/blob/master/provider/kavenegar/kavenegar.go
I added another comment for something small. Else looks good. Did you test it? |
Hi
I add ghasedak sms provider
main address is : https://ghasedak.me/