Skip to content
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

Don't crash when viewing in progress recording. #939

Merged
merged 9 commits into from
Jan 5, 2023
Merged

Don't crash when viewing in progress recording. #939

merged 9 commits into from
Jan 5, 2023

Conversation

jimdogx
Copy link
Contributor

@jimdogx jimdogx commented Jan 2, 2023

Current version crashes when trying to view a recording in progress.

Changes
Defensive coding

Issues
Fixes #501

source/VideoPlayer.brs Outdated Show resolved Hide resolved
source/VideoPlayer.brs Outdated Show resolved Hide resolved
source/VideoPlayer.brs Outdated Show resolved Hide resolved
source/VideoPlayer.brs Outdated Show resolved Hide resolved
jimdogx and others added 4 commits January 4, 2023 07:09
Co-authored-by: 1hitsong <3330318+1hitsong@users.noreply.github.com>
Co-authored-by: 1hitsong <3330318+1hitsong@users.noreply.github.com>
Co-authored-by: 1hitsong <3330318+1hitsong@users.noreply.github.com>
Co-authored-by: 1hitsong <3330318+1hitsong@users.noreply.github.com>
@jimdogx jimdogx requested a review from 1hitsong January 4, 2023 14:10
@jimdogx jimdogx merged commit 3c850dc into jellyfin:unstable Jan 5, 2023
@jimdogx jimdogx deleted the JF-501-Crash-when-watching-inprogress-recording branch January 5, 2023 13:37
@numericOverflow
Copy link

@jimdogx - Is there a way for me to test this updated version of the Roku app now that you've implemented a fix?

@jimdogx
Copy link
Contributor Author

jimdogx commented Jan 9, 2023

@numericOverflow if you feel comfortable with side loading apps on your Roku (it does not remove or alter your main installation), you can read up on it more here:

https://github.com/jellyfin/jellyfin-roku/blob/unstable/DEVGUIDE.md

Feel free to ask questions in the Matrix 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

app exits/crashes to home screen when trying to play recording still in progress
4 participants