Donavon West

Results 41 comments of Donavon West

> Changing to let/const should not be within the scope of such a change. Changing from `var` to `const` should _always_ be in scope ;) 💯 on the options object....

BTW, This was prompted by this Twitter thread. https://twitter.com/GabeRicard/status/1198316044957421568

I'm having the same issue. I just closed #5385 as it's a dupe of this issue. ~~If you need to temporary workaround (it works for me), you can use the...

@tarngerine Just to confirm, I can see your sandbox properly failing with 4.3.2

FYI... Here's the diff between 4.3.1 and 4.3.2 ## fetch.js ```diff diff --git a/src/fetch.js b/src/fetch.js index v4.3.1..v4.3.2 100644 --- a/src/fetch.js +++ b/src/fetch.js @@ -346,13 +346,8 @@ } }; - socket.prependListener('close',...

The only issue with reverting back to 4.3.1 is that 4.3.2 was there to fix a memory leak (see https://github.com/remix-run/web-std-io/commit/3b9b38446d186230a94d5481ba12ae4289d8809e).

PR created with a possible fix: https://github.com/remix-run/web-std-io/pull/29 Plz review

I'm surprised that something that crashes Remix isn't getting any attention.

I did a little poking around and found this package which could help https://github.com/AltSchool/dynamic-rest

Despite the name, this package has nothing to do with React. It's a JavaScript function that can be used with any client code framework/library.