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

v3.0: Guzzle 6 Support and rewrite #50

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

v3.0: Guzzle 6 Support and rewrite #50

wants to merge 12 commits into from

Conversation

rdohms
Copy link
Owner

@rdohms rdohms commented Oct 24, 2017

Working on getting this library updated to Guzzle 6, but this means a full rewrite.

  • Update definition generator to v6
  • Add support for Result models. This is new, it will replace our custom responses with nice deserialization models (hope it works for all endpoints, docs are shady).
  • Support Key Auth
  • Support OAuth 1.0
  • Support OAuth 2.0
  • Update docs
  • Migrate Rate Limit plugin

rdohms added 12 commits October 14, 2017 12:18
In order to get the latest methods and changes as per October 2017.
Now we can make use of Guzzle 6.
In order to no longer need to rely on response juggling, we are now
parsing the response from the API documentation, this will give us
nicely parse results using the new version of the library.
This will all be rewritten anyway.
With the new command that returns response models as well, now moved to
the root of the project for practicity.
In order to wrap Client and Server errors in easier to handle exceptions
Plugin plus configuration support to allows users to pass a key that
will be used in requests.
In order for each Config type to add some base configurations, the
interface now allows for setting those.
Added new Config model that cna pre-configure Oauth 1.0 auth, with
package suggestion in composer.json
Grant Types are configured outside the lib and our Config object hooks
its all up for easy peasy Oauth 2.
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