-
Notifications
You must be signed in to change notification settings - Fork 2
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
Enable public api #10
base: master
Are you sure you want to change the base?
Conversation
… the default options
… soon when v3 APIs are release
…PI url to be stored as this.newUrl in EnvoyAPI
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just noticed one of my commits breaks any app importing this SDK and deployed in Heroku due to an unexpected token error. I corrected this at 259ad68 and was able to confirm it fixed when testing with an app hosted on my Heroku.
Hello @anthonyFloccari and @johnmarrero, Would either of you be able to help review this PR? Let me know if this should be assigned to someone else to review and I'll reach out to them. |
Updated EnvoyAPI with additional methods to be consistent with the public APIs we have available.
Also added axios package. In the future we should update the requests to axios to move away from the nodeJS Request library which is now deprecated.