Skip to content

gemspec + delivery_method (+ Added Bearer to Authorization header) #6

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

AndyObtiva
Copy link

@AndyObtiva AndyObtiva commented Jun 9, 2017

I setup project for submitting to rubygem and generated a gemspec for it. This was done using Jeweler and Bundler. Also incorporated authorization bearer to fix issue with token used in API requests (as per #5). Thank you very much for providing this library. You should be able to submit as a rubygem by pulling this pull request into your local directory, merging, and running "rake release". Let me know if you need any help with this as I have released many gems in the past. I look forward to future versions of this library.

p.s. I tested the gem hosted on my github fork using this line in a project Gemfile:

gem 'sendpulse-rest-api-ruby', git: 'https://github.com/AndyObtiva/sendpulse-rest-api-ruby.git'

It worked! In the future, developers should be able to just reference the gem once released as follows:

gem 'sendpulse-rest-api-ruby'

or

gem 'sendpulse-rest-api-ruby', '~> 1.0.0'

Finally, a Rails ActionMailer delivery_method (:sendpulse) has been added to leverage the sendpulse-rest-api-ruby for email sending in Rails with minimal setup. Details are found in the updated README.md

@AndyObtiva AndyObtiva changed the title gemspec gemspec (+ Added Bearer to Authorization header) Jun 9, 2017
@AndyObtiva AndyObtiva changed the title gemspec (+ Added Bearer to Authorization header) gemspec + delivery_method (+ Added Bearer to Authorization header) Jun 9, 2017
@brunomperes
Copy link

@miguelpeniche
Copy link

Hey @AndyObtiva, is this still working?

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.

3 participants