Skip to content

Conversation

@e0d
Copy link
Contributor

@e0d e0d commented Dec 16, 2013

@jarv @polesye @dmitchell @jzoldak

Please review this updated PR and new branch. If things look good, I'll merge to master and create the release branch.

@dmitchell
Copy link
Contributor

👍

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this needs to be FEATURES not MITX_FEATURES

@polesye
Copy link
Contributor

polesye commented Dec 16, 2013

👍

@jzoldak
Copy link
Contributor

jzoldak commented Dec 16, 2013

lgtm 👍

@adampalay
Copy link
Contributor

@e0d , we don't want commit 76c8a7a

@stephensanchez
Copy link
Contributor

@waheedahmed @symbolist
There are a few changes in this merge around open ended grading, could you take a look and make sure this merge is correct according to your fixes on release?

@symbolist
Copy link
Contributor

👍

@e0d
Copy link
Contributor Author

e0d commented Dec 16, 2013

I've re-run the test that failed manually and it passed. This looks like an ephemeral issue.

https://jenkins.testeng.edx.org/job/edx-acceptance-manual-commit/53/console

e0d pushed a commit that referenced this pull request Dec 16, 2013
@e0d e0d merged commit 4de51c8 into master Dec 16, 2013
@e0d e0d deleted the e0d/release-merge branch December 16, 2013 22:08
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Jul 14, 2017
* change biz message openedx#1874

*  Implements bulk operation for biz openedx#1874
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.