fastify-multipart icon indicating copy to clipboard operation
fastify-multipart copied to clipboard

Outdated Example in README

Open is2ei opened this issue 3 months ago • 5 comments

Prerequisites

  • [x] I have written a descriptive issue title
  • [x] I have searched existing issues to ensure the issue has not already been raised

Issue

The README references Node.js EOL versions:

https://github.com/fastify/fastify-multipart?tab=readme-ov-file#parse-all-fields-and-assign-them-to-the-body

Request body in key-value pairs, like req.body in Express (Node 12+)

On Node 18+

These references should be updated.

is2ei avatar Sep 23 '25 08:09 is2ei

Hi! I’d like to work on this issue and update the README references to current Node.js versions. Could you please assign this to me? Thanks!

Jatin-2075 avatar Sep 23 '25 10:09 Jatin-2075

Hi! I’d like to work on this issue and update the README references to current Node.js versions. Could you please assign this to me? Thanks!

We don't assign Issues to avoid cookie licking.

Fdawgs avatar Sep 23 '25 11:09 Fdawgs

Thanks for the heads-up! Totally understand the no-assignment policy — just wanted to express interest and learn from contributing. Since this one's already being handled, I’ll explore other open issues and see where I can help. If there’s anything else in the pipeline that could use a hand, I’d love to jump in!

Jatin-2075 avatar Sep 23 '25 11:09 Jatin-2075

Can give it a go @Jatin-2075 if you want, the references just need removing. The original PR didn't resolve it.

Fdawgs avatar Sep 23 '25 17:09 Fdawgs

Thanks for reopening the issue! I’ll start working on removing the EOL Node.js references from the README and submit a PR shortly. Let me know if there are any specific sections you'd like me to focus on.

Jatin-2075 avatar Sep 24 '25 02:09 Jatin-2075