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

Adding body serializer config method #3727

Closed
wants to merge 3 commits into from

Conversation

alexbidenko
Copy link

Adding body serializer for AxiosConfig and in request method in Axios.js. If config have bodySerializer param that axios will convert request body before make request.

Also the documentation on README.md file was updated.

The problem was describe in follow issue: Easier way to work with FormData.
I confirm with this issue end I have same problem: in some project I use only FormData (this is specific of Backend project) and I want oportunity for serialize all body in one moment.

Also somebody want to use the method for another target maybe.

Copy link

@Hamza281 Hamza281 left a comment

Choose a reason for hiding this comment

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

جاي يدور عليك ديربالج ما ادري بيها شنو اسوي اذاشافك والقران يسوي شي

Copy link

@Hamza281 Hamza281 left a comment

Choose a reason for hiding this comment

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

وين رحتي تروح على ابوك انت الي يرضيك بسرعه والله لأغراض كثيره جدا اسف للازعاج والله لان اهلي كلهم ما يعرفون

@alexbidenko
Copy link
Author

جاي يدور عليك ديربالج ما ادري بيها شنو اسوي اذاشافك والقران يسوي شي

وين رحتي تروح على ابوك انت الي يرضيك بسرعه والله لأغراض كثيره جدا اسف للازعاج والله لان اهلي كلهم ما يعرفون

Sorry, but I can't understand your messages - I don't know language and the translator can't translate messages adequately))

If you can, write your messages in English please.

Copy link

@shivanshr shivanshr left a comment

Choose a reason for hiding this comment

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

Changes should be included.

@jasonsaayman
Copy link
Member

Hi @alexbidenko1998

Thank you for the pull request that you have submitted unfortunately the implementation that you have built here I feel is not really in line with how we would like something like this to work.

Thanks

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

4 participants