[ MITx - Theme 101 ] Move invitation(allowed_enroll) emails from edx_ace to Bulk emails templates #154
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
(cherry picked from commit ebf034b)
(cherry picked from commit ce098ad)
This is aggregating a collection of changes for adding a tab to the instructor dashboard for reporting on submissions to rapid response problems.
Trying to load the grade distribution for a given block is resulting in full table scans on the `courseware_studentmodule` table which contains over 15 million rows. This causes the page load to time out, as well as blocking other processes from accessing the table in a timely fashion. (cherry picked from commit 8b69081) (cherry picked from commit 787a215)
In order to allow for preventing users accidentally enrolling in a course we are adding a flag to toggle whether courses in a given installation of edx-platform are invite only, rather than relying on setting it at the per-course level. (cherry picked from commit 1be29e2) (cherry picked from commit 2581b511ceb44c6f6677ece4bf833c556fe766f0)
(cherry picked from commit 9987097)
Because CAS creates the user with `is_active` and `email` already set there is no change in those fields. By adding `date_joined` to the list of fields checked the auto enrollment will be processed properly. (cherry picked from commit 26307bf)
Updating edxval to use our modified version to fix transcript export
umarmughal824
changed the title
umarmughal824
changed the title
umarmughal824
changed the title
|
I don't understand what this PR is for? The link in the description goes to an issue about problems, not email. |
sorry, it was a typo, the description is updated now. |
blarghmatey
force-pushed
the
mitx/ironwood
branch
from
45c1e91 to
b79598d
Compare
blarghmatey
force-pushed
the
mitx/ironwood
branch
from
8f765c2 to
2f6cd65
Compare
|
@umarmughal824 this PR is really stale now and I'm not sure we even want to make changes to this mail template any more. I'm going to close this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
mitodl/mitx-theme#101