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

Init Channels support #24

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

Conversation

antoniogiordano
Copy link

I love this wrapper, i find it very useful.
It lacks of Channels support, that it's exactly what i need, so i'll try to add the support for it :-)

@@ -10,7 +10,7 @@ var AzureRequest = function ( config ) {
AzureRequest.prototype.buildConfig = function ( config ) {

this.config = config || {}
this.config.mediaURI = "https://media.windows.net/API/"
this.config.mediaURI = "https://media.windows.net/api/"
Copy link
Owner

Choose a reason for hiding this comment

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

I don't think this is correct. Docs maintain an uppercase API.

https://msdn.microsoft.com/en-us/library/azure/hh973617.aspx

@michelle-becker
Copy link
Owner

Wow, I apparently am horrible at checking on my old repos. I'm sorry this escaped my notice for the better part of the year! Thank you for the PR. I'd be happy to merge after the couple nitpicks and the removal of the .gitignore since that is really user specific.

@michelle-becker michelle-becker mentioned this pull request Aug 3, 2016
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.

2 participants