Skip to content

Disabled video autoplay for students - #1041

Merged
jmclaus merged 3 commits into
masterfrom
jmclaus/bugfix_disable_video_autoplay_lms
Sep 20, 2013
Merged

Disabled video autoplay for students#1041
jmclaus merged 3 commits into
masterfrom
jmclaus/bugfix_disable_video_autoplay_lms

Conversation

@jmclaus

@jmclaus jmclaus commented Sep 18, 2013

Copy link
Copy Markdown

The autoplay functionality is turned off now. Should we just remove it altogether?

@ghost ghost assigned auraz Sep 18, 2013
@jmclaus

jmclaus commented Sep 18, 2013

Copy link
Copy Markdown
Author

@auraz @polesye Please review.

@polesye

polesye commented Sep 18, 2013

Copy link
Copy Markdown
Contributor

Were acceptance tests for lms passed (rake test_acceptance_lms["lms/djangoapps/courseware/features/video.feature"])?

@jmclaus

jmclaus commented Sep 18, 2013

Copy link
Copy Markdown
Author

@polesye A good point. Acceptance tests for LMS now pass.

@polesye

polesye commented Sep 19, 2013

Copy link
Copy Markdown
Contributor

👍

@polesye

polesye commented Sep 19, 2013

Copy link
Copy Markdown
Contributor

@jmclaus please rebase your branch on master.

@jmclaus

jmclaus commented Sep 19, 2013

Copy link
Copy Markdown
Author

@polesye Done. I'm waiting for @auraz comments and I will merge.

@auraz

auraz commented Sep 20, 2013

Copy link
Copy Markdown
Contributor

👍

jmclaus pushed a commit that referenced this pull request Sep 20, 2013
…ay_lms

Disabled video autoplay for students
@jmclaus
jmclaus merged commit 1c7d821 into master Sep 20, 2013
@jmclaus
jmclaus deleted the jmclaus/bugfix_disable_video_autoplay_lms branch September 20, 2013 12:06
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Jul 28, 2016
…-biz-playback-status-batch

Add update biz playback status batch openedx#1039
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Aug 9, 2016
…-biz-playback-status-batch

Add update biz playback status batch openedx#1039
(cherry picked from commit af250e4)
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Aug 9, 2016
Merge pull request openedx#1041 from yokose-ks/develop/cypress/update-biz-pl…
afzaledx pushed a commit to open-craft/openedx-platform that referenced this pull request Jun 1, 2018
…ion-bumps

extension apps version bumps in custom.txt
ericfab179 added a commit to eduNEXT/edx-platform that referenced this pull request Apr 5, 2019
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