Skip to content

form type 'x-mixed-replace' #541

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

Closed
amitrohatgi opened this issue Jan 11, 2020 · 8 comments
Closed

form type 'x-mixed-replace' #541

amitrohatgi opened this issue Jan 11, 2020 · 8 comments

Comments

@amitrohatgi
Copy link

is 'multipart/x-mixed-replace' supported? If not, I'm happy to contribute, but a couple of pointers would be helpful!

@GrosSacASac
Copy link
Contributor

It is not, I need to push my branch and then we can add more parsers which are transform streams

@tunnckoCore
Copy link
Member

tunnckoCore commented Jan 29, 2020

@amitrohatgi the streams branch of @GrosSacASac is merged into master. We can continue with this.

Pointers:

  1. make a file in src/parsers
  2. make a class extending stream.Transform
  3. use module.exports to export the class, module.exports = JSONParser;

@tunnckoCore
Copy link
Member

@all-contributors please add @amitrohatgi for ideas

@allcontributors
Copy link
Contributor

@tunnckoCore

I've put up a pull request to add @amitrohatgi! 🎉

@tunnckoCore
Copy link
Member

again... @all-contributors please add @amitrohatgi for ideas

@allcontributors
Copy link
Contributor

@tunnckoCore

I've put up a pull request to add @amitrohatgi! 🎉

@tunnckoCore
Copy link
Member

@amitrohatgi #545 Plugins API is going to master, so you can start :)

@GrosSacASac
Copy link
Contributor

Closing, but feel free to make a PR

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

No branches or pull requests

3 participants