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

Replace Buzz with Guzzle #23

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

Conversation

Jontsa
Copy link
Contributor

@Jontsa Jontsa commented Mar 30, 2018

Buzz 0.17 broke backwards compatibility and would have required either a version constraint or code to be updated to match. Here instead code is updated to use Guzzle which has some cool features and is more widely used.

  • use Guzzle HTTP client instead of Buzz
  • updated tests to match and make use of Guzzle mock handler
  • removed carriage returns in some files and fixed formatting
  • updated Infobip gateway to use newer JSON API

All tests pass. Sending an actual message was only tested using InfobipGateway.

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.

1 participant