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

Hotfix/release multiple video bug #850

Closed
wants to merge 6 commits into from

Conversation

auraz
Copy link
Contributor

@auraz auraz commented Sep 3, 2013

This PR addresses the issue BLD-292: "If multiple video players are put on the same page, only one of them works".

nedbat and others added 6 commits August 28, 2013 10:15
Comments (and processing instructions!) are handled oddly in lxml.
This change will keep them from causing failures.  They will be omitted
from the HTML generated, which is fine, since they aren't needed there.
Check extension rather than mimetype
@auraz auraz closed this Sep 3, 2013
@polesye polesye deleted the hotfix/release-multiple-video-bug branch February 25, 2014 09:24
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
Adding django-admin/rake command to set the staff bit
ny0m pushed a commit to open-craft/edx-platform that referenced this pull request Aug 21, 2017
andrey-canon pushed a commit to eduNEXT/edx-platform that referenced this pull request Aug 16, 2018
caesar2164 pushed a commit to caesar2164/edx-platform that referenced this pull request Dec 11, 2018
…_report

Add timestamp to Get Student Responses Report
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.

4 participants