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

Sync up master with changes in dev #101

Merged
merged 81 commits into from
Aug 31, 2018
Merged

Sync up master with changes in dev #101

merged 81 commits into from
Aug 31, 2018

Conversation

muthurathinam
Copy link
Contributor

No description provided.

Colby Sieber and others added 30 commits November 14, 2017 17:08
Export all, remove package.json code dependency
Requests are now processed using the fetch API
Response Handler needs some refactoring to handle the error vs bad response
Added configuration for running core tests from a profile within VS Code.
Merge pull request #69 from microsoftgraph/testRunConfig
Switch to isomorphic fetch
User headers now overwrite default headers
Correct the parameter to accept array of type string
expand, orderby methods.
2. Fixed compilation error by adding required libraries in config.
Fixed Compilation error by changing property value param to accept array of strings in select, expand, orderby methods and by adding required libraries in config.
Changed [string] to string[], fixed spacing, fixed double parens
Use the Fetch API instead of SuperAgent to communicate with MS Graph
muthurathinam-m and others added 19 commits August 8, 2018 16:15
2. Pull request changes

This reverts commit a72448b.
This reverts commit 92c4eda.
Fix request-id of undefined error in graph error object creation in request handler
…ad-As-Binary

Have enum for ResponseType and Update README.md with download as binary example
Create two different output js files on with vendor polyfills and one without
…OneDriveLargeFileUploadTask, write corresponding unit tests
Copy link

@deepak2016 deepak2016 left a comment

Choose a reason for hiding this comment

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

As this is a merge from dev to master, approving this merge. AS pull request is long, not practical to go through every file and line of change.

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.

None yet

7 participants