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

data param should allow <String, dynamic> #111

Closed
Dheena32 opened this issue Oct 23, 2020 · 1 comment · Fixed by #127
Closed

data param should allow <String, dynamic> #111

Dheena32 opened this issue Oct 23, 2020 · 1 comment · Fixed by #127
Assignees
Labels
enhancement New feature or request

Comments

@Dheena32
Copy link

As of now data param allowing only <String, String>
Please add provision for allowing <String, dynamic>
so that we can send multiple level of json format like ,
{ product: {
name: ['Apple', 'Orange']
}
}

@ened
Copy link
Collaborator

ened commented Oct 23, 2020

@Dheena32 I'm thinking to replace the data parameter with a generic ByteList, so that you could send anything you'd like. The uploader is currently forcing JSON but it should really be up to the API user. Thx for the feedback.

@ened ened self-assigned this Oct 23, 2020
@ened ened added the enhancement New feature or request label Oct 23, 2020
@ened ened closed this as completed in #127 Dec 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants