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

Workaround for issue #701

Merged
merged 1 commit into from
Aug 15, 2013
Merged

Conversation

JonahStanley
Copy link
Contributor

Suggested Reviewers @wedaly @jzoldak

Workaround for an issue with setting USE_I18N = False for acceptance tests
It prevents courseware.courses from being imported on the lms acceptance tests. This is purely a work around for now and we can circle back to it later. This only affects the acceptance tests

Better wording of commit

Workaround for issue

Better wording of commit

Fixed spacing
@jzoldak
Copy link
Contributor

jzoldak commented Aug 15, 2013

👍

JonahStanley pushed a commit that referenced this pull request Aug 15, 2013
@JonahStanley JonahStanley merged commit 796667c into master Aug 15, 2013
@JonahStanley JonahStanley deleted the jonahstanley/fix-lms-acceptance-issue branch August 15, 2013 21:01
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
Trivial fix to detailed-solution style
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Apr 19, 2016
caesar2164 pushed a commit to caesar2164/edx-platform that referenced this pull request Oct 24, 2017
* stv/break-the-teapot:
  Revert "Allow shibboleth via logistration"
andrey-canon pushed a commit to eduNEXT/edx-platform that referenced this pull request May 30, 2018
…ailure-admin

Proversity/login failure admin
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
It doesn't seem to be working - instead never calling ecommerce and
always showing the normal non-discount version of the streak modal.

Going to investigate later, but this is just a quick shutoff for
now.
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
openedx#706)

This reverts commit f93519f.

The original fix may have been working after all and a bogus
test by me using a staff account made me think the discount was
not being offered. Reverting the emergency fix and going to test
the original fix again.
johanseto pushed a commit to nelc/edx-platform that referenced this pull request Jan 22, 2024
feat: Add email field to UserMappingView
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