-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add notification on playback error #109
Conversation
👎 @felixheidecke I cannot get the new notification: Tests steps from #35 (comment) and Chrome for Mac Version 76.0.3809.100 Console error was spotted
|
@davitol are you sure you compiled? Becauuuuuse … the console error is from … before :-P |
@felixheidecke Retested and keeps not working for me. And I get the log:
Tested in Chrome for Mac Version 76.0.3809.100 and oC 10.3 alpha |
c534533
to
6432da7
Compare
@davitol looks like I forgot to add this notification for images as well 😅 |
a96d7ca
to
e038b27
Compare
e038b27
to
9cdbd25
Compare
@davitol can you have another look? I think that'll fix it. |
@davitol ping |
Ping... any progress ? |
Tested and LGTM
|
BTW Warning Fail to load video data is not translated |
Add notification if video playback fails.
Fixes #35