Skip to content

Conversation

stevenbrookes
Copy link

Hi Tareq

As discussed see what you think of this. We are still running some testing but looks good so far,.

Regards

Steve

@shawnhind
Copy link

I think the API versioning should be set at the config level. It's possible that for many API version updates that the library will not need to be updated but that the data that gets returned could change.

Perhaps specify versions of the API that have been tested with the library and default to 2019-04 but allow setting version through the config.

Copy link

@VarunBatraIT VarunBatraIT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

public static $version = '2019-04';

It should be protected and should have a getter and setter

@tareqtms
Copy link
Contributor

@stevenbrookes
I have implemented it in a different way. Added it in $config Array, key ApiVersion. So it can be customized by users through $config when needed.
Please check commit a1fcdec

@stevenbrookes
Copy link
Author

stevenbrookes commented May 24, 2019 via email

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

Labels

4 participants