Pull Request to add outbound SMS notifications to Broadcast #6

Merged
merged 1 commit into from Jun 11, 2011

Projects

None yet

2 participants

@chrismatthieu
Contributor

Added SMS medium using the SMSified gem to allow Broadcast users to send SMS messages to any US mobile device.

Developers must create an account on http://smsified.com before sending SMS text messages (Developer accounts are free). You will be given a phone number to use as your very own FROM sms number. This number, along with your username and password, must be added to your config during setup. The To address is the address of the mobile number that you would like to send the SMS message to.

Example setup

  Broadcast.setup do |config|
    config.Sms { |sms|
      sms.username       = 'myaccount'
      sms.password       = 'mypass'
      sms.from           = '16025551212'
      sms.to             = '14801234567'
    }
  end
@chrismatthieu
Contributor

This pull request addresses issue number 2 that I opened in your repo. Thanks!

@marcinbunsch marcinbunsch merged commit c1a36f6 into futuresimple:master Jun 11, 2011
@marcinbunsch
Contributor

Awesome, thanks a lot for this! I saw that your medium inherits from Broadcast::Medium::Oauth, but then I looked into the code and realized that 5 media do that even though they do not use OAuth! No worries though, I took care of that :)

I also added the smsfied gem to the gemspec and a simple spec for regression testing,

@chrismatthieu
Contributor

Perfect. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment