Skip to content

fix: video crashes - #520

Merged
volodymyr-chekyrta merged 1 commit into
openedx:developfrom
edx:upstream-develop-crashfix-4
Sep 30, 2024
Merged

fix: video crashes#520
volodymyr-chekyrta merged 1 commit into
openedx:developfrom
edx:upstream-develop-crashfix-4

Conversation

@rnr

@rnr rnr commented Sep 6, 2024

Copy link
Copy Markdown
Contributor

App was crashing in two cases of videos, both the crashes can be tested on course Project Management for Development

  1. When the LMS is returning an empty URL for video encoding

How it can be tested:
It can be tested on the first section, Start here -> 0.1 Welcome to the course -> going to the video
Solution:
Mimic the behaviour from the old app.
2. When the LMS returns wrong data for transcripts, LSM is returning less time for endtime than the starttime of the transcript.

How it can be tested:
1. Go to Videos tab
2. Go to Module 1: Project Chapter
3. Go to 1.1.5 Video: Project Chapter (can be selected from the dropdown switcher)
4. Go to the video component

@volodymyr-chekyrta
volodymyr-chekyrta merged commit d5a55fa into openedx:develop Sep 30, 2024
@rnr
rnr deleted the upstream-develop-crashfix-4 branch October 7, 2024 10:21
IvanStepanok pushed a commit to raccoongang/openedx-app-ios that referenced this pull request Sep 1, 2025
Co-authored-by: Saeed Bashir <sbashir@2u.com>
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.

3 participants