Skip to content

Revert "fix: bundle video in app to prevent crash when not available"#5940

Merged
NicolasMassart merged 1 commit intomainfrom
revert-5886-471_make_SRP_video_embeded_in_app
Mar 9, 2023
Merged

Revert "fix: bundle video in app to prevent crash when not available"#5940
NicolasMassart merged 1 commit intomainfrom
revert-5886-471_make_SRP_video_embeded_in_app

Conversation

@NicolasMassart
Copy link
Copy Markdown
Contributor

Reverts #5886
Because current apps are relying on the video hosted on main branch and this PR renamed the video to add source in name making it fail to load and displays a black square instead of the video.
We have to apply the PR again with taking this retro compatibilty into account.

@NicolasMassart NicolasMassart requested a review from a team as a code owner March 9, 2023 17:14
@NicolasMassart NicolasMassart requested a review from sethkfman March 9, 2023 17:14
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 9, 2023

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

Copy link
Copy Markdown
Contributor

@Cal-L Cal-L left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@sethkfman sethkfman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@NicolasMassart NicolasMassart merged commit c4c2313 into main Mar 9, 2023
@NicolasMassart NicolasMassart deleted the revert-5886-471_make_SRP_video_embeded_in_app branch March 9, 2023 17:26
@github-actions github-actions bot locked and limited conversation to collaborators Mar 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants