Skip to content

Enable per-student background tasks. - #254

Merged
brianhw merged 1 commit into
masterfrom
fix/brian/rescore-cleanup
Jun 24, 2013
Merged

Enable per-student background tasks.#254
brianhw merged 1 commit into
masterfrom
fix/brian/rescore-cleanup

Conversation

@brianhw

@brianhw brianhw commented Jun 24, 2013

Copy link
Copy Markdown
Contributor
  • Make per-student background task buttons appear on the instructor dash, by using the correct name of the MITX_FEATURE (i.e. ENABLE_INSTRUCTOR_BACKGROUND_TASKS).
  • Remove redundant transaction.autocommit annotation in api_helper.py, since that is handled in save_now().
  • Add docstrings.
  • Add two tests that will work when Mako templates are reliably initialized for tests.

@ghost ghost assigned dianakhuang Jun 24, 2013
@brianhw

brianhw commented Jun 24, 2013

Copy link
Copy Markdown
Contributor Author

@dianakhuang @nedbat, if you please....

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we put this indentation back the way it was?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

My bad. I fixed the broken indentation independently of Cale, not realizing he had done so and doing it in a different way. Yes, I'll fix.

@dianakhuang

Copy link
Copy Markdown
Contributor

Are you going to squash the second commit? I think you should either do that or fix the commit message.

@dianakhuang

Copy link
Copy Markdown
Contributor

Other than that, 👍

brianhw added a commit that referenced this pull request Jun 24, 2013
Enable per-student background tasks.
@brianhw
brianhw merged commit 3f2d33c into master Jun 24, 2013
@brianhw
brianhw deleted the fix/brian/rescore-cleanup branch June 24, 2013 22:49
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
Kelketek referenced this pull request in open-craft/openedx-platform Oct 21, 2014
ziafazal/api-remove-rounding: remove round in progress
CrewS pushed a commit to CrewS/edx-platform-1 that referenced this pull request Jan 17, 2019
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
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.

3 participants