-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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.
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
Error UI block RCTVideo 0x123e3f990 #907
Comments
Also seeing this issue +1 |
@JayricMok and @kvieira90 this commit: svhawks@511c73e fixes this. I think @ardalahmet will create a PR ? I just tested that and it works. Me and @rclements were having this issue. |
Hi @franzejr If I end up with a better fix I'll create a PR for sure. Thanks. |
Has this been merged? |
Also seeing this issue +1 |
Seeing this. Is there a reason not to merge ? |
We are experiencing this issue as well |
We have confirmed that this PR has fixed the problem |
@ardalahmet could you please open a pull request for this fix? |
When are you seeing this? I've tried switching between HLS streams and am not encountering any issues. I want to see if I can figure out why we don't have an observer to remove, if I can't I'll merge this fix. Either way I'll get it fixed right away. |
Ok, didn't know exactly how to check this, but #1075 should handle it. It felt a little bit cleaner to use a boolean to track if there was an observer to clear so that we match the code used for the other observers. Thanks to everyone who submitted a PR. |
This reverts commit 343807e.
I getting two error when I switch m3u8 link in iOS, Android Work Perfectly.
Exception thrown while executing UI block: Cannot remove an observer <RCTVideo 0x123e3f990> for the key path "readyForDisplay" from <AVPlayerLayer 0x1c4625060> because it is not registered as an observer.
Exception thrown while executing UI block: 'parentNode' is a required parameter
The text was updated successfully, but these errors were encountered: