Pooya Parsa comments

Results 2282 comments of


                                            Pooya Parsa

feat(options): add allowMultiple to options

Thanks for PR. I am still not sure which method to go but am more towards last option (of joining with `, `). because of this edge case, all code...

Missing "./driver/localstorage" specifier in "unstorage" package

It is probably a typo `unstorage/drivers/localstorage` is correct (with s). Dos: https://unstorage.unjs.io/drivers/browser#local-storage

refactor!: remove `ufo` dependency

It is planned for next major version alongside other dep drops (like node-fetch-native and node < 20)

GraphQL Support

as much as tempting as the idea is, i think it is out of ofetch scope (could be a wrapper library that does the variable replacement, and more)

Deno support

Probably related (https://github.com/denoland/deno/issues/26700, https://github.com/denoland/deno/pull/23289)

Unexpected token export declare

Please provide a reproduction 🙏🏼

fix(dev): apply headers from route rules for static assets

Added tests, two issues: - `path` passed inside middleware omits base `/build/test.txt` gets `/test.txt` therefore rules isn't matching. (we need to find solution) - Currently, nitro has an implicit behavior...

fix(dev): apply headers from route rules for static assets

There is a clear description of the state: https://github.com/nitrojs/nitro/pull/2814#issuecomment-2463082441

ENOENT: no such file or directory, access '/tmp/nitro/worker-..sock'

A relevant fix recently landed https://github.com/nitrojs/nitro/pull/2997 If anyone encounters it, can you try [nitro nightly channel](https://nitro.build/guide/nightly) if it fixes?

Make server routes aware of page routes to avoid collisions

I would suggest follow-up via #25709 (https://github.com/nuxt/nuxt/issues/25709#issuecomment-2345577725https://github.com/nuxt/nuxt/issues/25709#issuecomment-2345577725), thanks for nice writeup dear @ChrisGV04.