Skip to content

Mondido PHP SDK 2.0 #3

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 36 commits into
base: master
Choose a base branch
from
Open

Mondido PHP SDK 2.0 #3

wants to merge 36 commits into from

Conversation

phroggyy
Copy link

  • Rewrote the SDK to use composer for autoloading and usage of PHPUnit in dev
  • Modified the repository to follow PSR-0 convention for file tree structure, and PSR-2 for code style
  • Made the SDK non-static to allow for dynamically defining username, secret, and other account-specific variables

Example usage of the new SDK can be found in the readme.

This is a breaking update which is not backwards-compatible with 1.1

phroggyy added 30 commits July 21, 2015 10:35
…ty with the api even if the user makes a mistake
…in the constructor, so that it is available to all our api classes, and referenced the change in the api classes. Also made all methods non-static as to work with with the new concept of a non-static SDK, with the Mondido object being instantiated from the beginning.
…s the api with a given username, password and secret. Also added some helper methods which allows the user to access a few of the private attributes on the instance. Also added access methods to access the different apis: transaction, refund and storedCard
…lowed attributes to ensure that we don't get any data which shouldn't be there. When creating a credit card, you can now also pass aliases, e.g 'cvv' instead of 'card_cvv'
…cally injecting user and pass. Also added a method to retrieve a single subscription
@adevade
Copy link

adevade commented Feb 16, 2017

Is this still being worked on?

Would love to load the SDK with Composer...

@emilsundberg
Copy link

@adevade We converted this to an internal package in our project since we do not currently have the possibility to support an unofficial open source version.

@olegisk olegisk mentioned this pull request Jan 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants