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

Added JavaScript version #3

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

Added JavaScript version #3

wants to merge 40 commits into from

Conversation

shmuelie
Copy link
Contributor

Its not perfect but...

@shmuelie
Copy link
Contributor Author

One big advantage of using a REST based implementation is that you can access the API end points in browser too. This JS version is designed to make doing that easier as well as allow developers to switch between C# and JavaScript quickly since the JS versions API is very close (though not identical) to the C# version.

While built for Web Apps the code may work in WinJS Apps too (not tested though)

…rsion that limited JS needs for requests relative to host, a non-need in C#
… calls the callback with the response as text and the error as a third parameter
@robertmclaws
Copy link
Collaborator

Hey Samuel,

If you could take some time with the current release and port it to WinJS for Windows 8.1 apps, I think that would be a really important use-case that we're not currently covering. With Microsoft's new license changes for Mono, WinJS is the only .NET platform where we don't currently have coverage.

Thanks man!

@shmuelie
Copy link
Contributor Author

shmuelie commented Dec 2, 2013

Currently playing catchup after going on a 2 week "break from the world" but I'll give I will give it a look. Off the top my head though it should work with zero changes or at least very little.

@shmuelie
Copy link
Contributor Author

Did some testing and it passed them all so I say good to go

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