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

add Ghasedak Sms Provider #101

Merged

Conversation

vessaldaneshvar
Copy link
Contributor

Hi
I add ghasedak sms provider
main address is : https://ghasedak.me/

// Retrieving required data from 'ghasedak' sections of config.yaml
type Config struct {
APIToken string `yaml:"api_token"`
PhoneNumbers []string `yaml:"phone_numbers"`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PhoneNumbers is not used!

Copy link
Contributor

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

Copy link
Contributor Author

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

Copy link
Contributor

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

Copy link
Contributor Author

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@marcelcorso
Copy link
Contributor

I added another comment for something small. Else looks good. Did you test it?

@marcelcorso marcelcorso merged commit ca9ca58 into messagebird:master Dec 8, 2021
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.

2 participants