Skip to content

Celery task for SOL-174 - #7158

Closed
marjev wants to merge 2 commits into
masterfrom
mjevtic/SOL-174
Closed

Celery task for SOL-174#7158
marjev wants to merge 2 commits into
masterfrom
mjevtic/SOL-174

Conversation

@marjev

@marjev marjev commented Feb 27, 2015

Copy link
Copy Markdown
Contributor

@martynjames Here's the work Davorin did for creating a celery task to do the indexing once the changes in course are published.

This is just a review PR, still not ready for merging.

dsego and others added 2 commits February 27, 2015 10:04
Conflicts:
	common/lib/xmodule/xmodule/modulestore/mongo/draft.py
@openedx-webhooks

Copy link
Copy Markdown

Thanks for the pull request, @marjev! It looks like you're a member of a company that does contract work for edX. If you're doing this work as part of a paid contract with edX, you should talk to edX about who will review this pull request. If this work is not part of a paid contract with edX, then you should ensure that there is an OSPR issue to track this work in JIRA, so that we don't lose track of your pull request.

To automatically create an OSPR issue for this pull request, just visit this link: http://openedx-webhooks.herokuapp.com/github/process_pr?number=7158&repo=edx%2Fedx-platform

1 similar comment
@openedx-webhooks

Copy link
Copy Markdown

Thanks for the pull request, @marjev! It looks like you're a member of a company that does contract work for edX. If you're doing this work as part of a paid contract with edX, you should talk to edX about who will review this pull request. If this work is not part of a paid contract with edX, then you should ensure that there is an OSPR issue to track this work in JIRA, so that we don't lose track of your pull request.

To automatically create an OSPR issue for this pull request, just visit this link: http://openedx-webhooks.herokuapp.com/github/process_pr?number=7158&repo=edx%2Fedx-platform

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This file has a number of pylint violations

@openedx-webhooks

Copy link
Copy Markdown

Thanks for the pull request, @marjev! It looks like you're a member of a company that does contract work for edX. If you're doing this work as part of a paid contract with edX, you should talk to edX about who will review this pull request. If this work is not part of a paid contract with edX, then you should ensure that there is an OSPR issue to track this work in JIRA, so that we don't lose track of your pull request.

To automatically create an OSPR issue for this pull request, just visit this link: http://openedx-webhooks.herokuapp.com/github/process_pr?repo=edx%2Fedx-platform&number=7158

@martynjames

Copy link
Copy Markdown

Closing for now, this work will eventually be incorporated into https://github.com/edx/edx-platform/pull/7313

@marjev
marjev deleted the mjevtic/SOL-174 branch May 28, 2015 14:39
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