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

expose some superagent options as properties for UploadManager #9

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

Conversation

digital-flowers
Copy link

i was testing your component with azure cloud Shared Access Signature (SAS) and i faced some problems and i solved them as follow:
1- add method property default as post, some cloud storage required put instead of post
2- add accept property default asapplication/json, if response doesn't return json for example using cloud SAS this property can cause a problem
3- add field name property default to file, when user use cloud SAS user can set property to null to send the file without form
4- set request type from file.type

@lionng429
Copy link
Owner

@digital-flowers Thanks for your PR. could you please make changes on just the files in /src instead of /lib? we should always work on the source codes but not compiled codes. thoughts?

@digital-flowers
Copy link
Author

i saw many react libraries include the lib files in the source code so some users can do npm install directly from github what do you think ?!

lionng429 added a commit that referenced this pull request Mar 22, 2018
- added `reqConfigs` as a prop which accepts `accept`, `method`, `timeout` and `withCredentials` properties
- added `formDataParser` as a prop which allows custom fields to be sent through in the request
- referred to the suggestions in #9
@lionng429 lionng429 mentioned this pull request Mar 22, 2018
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