Cory Johnson

Results 14 comments of Cory Johnson

Hi @markandrus - wanted to check in and see if you needed any more info / anything else from us for this PR at this time? I noticed the tests...

@mlogan / @markandrus FWIW, we are also running into this bug, also in calling GetTransceivers; in our case it appears to also be a race condition when trying to connect,...

Hi @markandrus / @mlogan / @dxiao2003 - I've created a PR, https://github.com/node-webrtc/node-webrtc/pull/667, which fixes the second issue mentioned here. I'm not 100% confident if it is the correct way to...

The following patch "works" for me locally, but I'm not 100% confident if it's "correct", so I'm holding off on a full PR at the moment: ``` diff --git a/package.json...

Just chiming in here to echo that this is actively affecting our Expo 51 project as well, and we're having to workaround it ATM by committing everything (which won't be...

@natew Adding my latest notes on this after spending 6+ hours on it today trying to dial things in a bit more from @GibbyBox 's example above. What I have...

One more follow-up; as I was closing out tabs from the investigation I was looking at Metro docs one more time, and caught this: https://metrobundler.dev/docs/resolution/#doesfileexist-string--boolean ``` By default, Metro implements...

Just to clarify @natew - if we are using both web and native (which we are / is critical for us), is removing the metro-plugin still viable? Can you clarify...

Just adding our :+1: / request for an update on this PR to the list! We would definitely benefit from pulling this in.

+1 - this is a major issue for us as well and we would expect it given that Amplitude, Mixpanel, and others all have clear ways to accomplish this goal...