Skip to content

Show notification on course outline reorder. - #540

Merged
peter-fogg merged 1 commit into
masterfrom
peter-fogg/overview-reorder-notification
Aug 1, 2013
Merged

Show notification on course outline reorder.#540
peter-fogg merged 1 commit into
masterfrom
peter-fogg/overview-reorder-notification

Conversation

@peter-fogg

Copy link
Copy Markdown
Contributor

@cahrens

cahrens commented Jul 31, 2013

Copy link
Copy Markdown

👍 I checked out out the branch and tested it manually. I did not try running the acceptance test.

@dmitchell

Copy link
Copy Markdown
Contributor

👍 after tests pass

@chrisndodge

Copy link
Copy Markdown
Contributor

master is unstable in terms of pep8 violations. @wedaly sayz OK not to wait on merges to get master back under pep8 violation limit.

@wedaly

wedaly commented Jul 31, 2013

Copy link
Copy Markdown
Contributor

A PR got merged at the end of today to fix pep8/pylint in master; if you merge or rebase with master, your build should pass.

@peter-fogg

Copy link
Copy Markdown
Contributor Author

Rebased; I'll wait till tests pass just to be safe.

peter-fogg pushed a commit that referenced this pull request Aug 1, 2013
…tion

Show notification on course outline reorder.
@peter-fogg
peter-fogg merged commit e501a17 into master Aug 1, 2013
@peter-fogg
peter-fogg deleted the peter-fogg/overview-reorder-notification branch August 1, 2013 14:18
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
…_syncing

getting rid of terrible hack in favor of re enabling the dispatcher to sync w/ the comment service
e-kolpakov referenced this pull request in open-craft/openedx-platform Oct 28, 2015
…r_email_message

Changed error message to be more in-line with possible reasons of the problem
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Jun 15, 2016
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Aug 9, 2016
* Merge pull request openedx#937 from nttks/gacco/cypress.biz-2nd (openedx#943)

* Merge pull request openedx#937 from nttks/gacco/cypress.biz-2nd

gacco/cypress.biz 2nd
(cherry picked from commit 4bc3bc2)

Conflicts:
	biz/djangoapps/ga_achievement/management/commands/tests/test_update_biz_score_status.py
	lms/djangoapps/ccx/tests/test_field_override_performance.py
	lms/djangoapps/instructor/tests/test_api.py
	lms/envs/test.py
	setup.py

* Merge pull request openedx#941 from kawaguchi-ks/develop/cypress/biz-fix-timezone-problem

Fix timezone problem on boundary test.
(cherry picked from commit 20e1b0c)

* Fix transaction, autocommit(commit_on_success) -> atomic.

* Define class Meta.

* Change AUTHENTICATION_BACKENDS for disable RateLimitBackend within lo… (openedx#902)

* Change AUTHENTICATION_BACKENDS for disable RateLimitBackend within login authentication(openedx#540) (openedx#883)
(cherry picked from commit 1e8b68a)

Conflicts:
	cms/djangoapps/course_creators/tests/test_admin.py

* Fix test, "RedirectCycleError: Redirect loop detected".
caesar2164 pushed a commit to caesar2164/edx-platform that referenced this pull request Nov 30, 2016
…bmit-fix

Update free text response xblock SHA for Enum value fix
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.

5 participants