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.

Sign up for GitHub

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

Jump to bottom

Option to ignore empty arrays #47

Closed
Sanva opened this issue Sep 10, 2023 · 6 comments
Closed

Option to ignore empty arrays #47

Sanva opened this issue Sep 10, 2023 · 6 comments

Comments

@Sanva
Copy link
Contributor

Sanva commented Sep 10, 2023

Hi!

I'm replacing some code using jQuery ajax method, and I find your package very useful for this endeavor — thanks a lot!

Would you consider adding an option to skip empty arrays? I can do it myself and make a PR if you want.

Best regards! :)
@iambumblehead
Copy link
Owner

Yes let's support skipping empty arrays. If you don't make a PR, I could make one tomorrow.

@iambumblehead
Copy link
Owner

ignoreEmptyArray is published to version 6.1.1 https://www.npmjs.com/package/form-urlencoded

thank you! :)

@Sanva
Copy link
Contributor Author

Sanva commented Sep 11, 2023

By the way, now there is an error in form-urlencoded.d.ts, it still has the prop in plural form.

@iambumblehead
Copy link
Owner

ok thanks will publish it a fix as soon as possible

@iambumblehead
Copy link
Owner

#52

@Sanva
Copy link
Contributor Author

Sanva commented Sep 11, 2023

Thanks! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Sanva @iambumblehead