Skip to content

Fix for acceptance test static file issue - #849

Merged
wedaly merged 1 commit into
masterfrom
will/acceptance-test-static-files
Sep 3, 2013
Merged

Fix for acceptance test static file issue#849
wedaly merged 1 commit into
masterfrom
will/acceptance-test-static-files

Conversation

@wedaly

@wedaly wedaly commented Sep 3, 2013

Copy link
Copy Markdown
Contributor

Recent changes to how we start up django apps broke lettuce tests.

  1. Previously, we replaced django.contrib.staticfiles with django-staticfiles to satisfy both lettuce (uses the built-in library) and mitxmako (uses external library).

  2. Changes to how we initialize Django apps caused django.contrib.staticfiles.handlers to no longer point to django-staticfiles.handlers.

  3. This PR replaces django.contrib.staticfiles.handlers explicitly to avoid this issue.

@jzoldak

@jzoldak

jzoldak commented Sep 3, 2013

Copy link
Copy Markdown
Contributor

👍

wedaly pushed a commit that referenced this pull request Sep 3, 2013
Fix for acceptance test static file issue
@wedaly
wedaly merged commit a081746 into master Sep 3, 2013
@wedaly
wedaly deleted the will/acceptance-test-static-files branch September 3, 2013 16:51
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
ny0m referenced this pull request in open-craft/openedx-platform Aug 21, 2017
Problem builder version bump to 2.7.2
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
* stv/fix/library:
  Move library creator checks to POST-only
  Revert "Revert "Merge pull request openedx#16447 from appsembler/omar/hide-library-button""
  Revert "Merge pull request openedx#16447 from appsembler/omar/hide-library-button"
  Hide the create library for non-staff users when DISABLE_COURSE_CREATION
DanielVZ96 referenced this pull request in open-craft/openedx-platform Jan 31, 2024
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.

2 participants