Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Disabled MITx from program nudge emails #36071

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

irfanuddinahmad
Copy link
Contributor

Description

This PR disables the program nudge emails for MITx

Supporting information

Exempt MIT courses from the send_program_course_nudge emails that are designed to go out to a learner after they have earned a passing grade in a course for 2 reasons:

  1. Timing - these are being sent out before their course end dates have passed AND before certificate display availability dates have passed.
  2. Inaccurate recommendations - one of their program series has a set sequence, and this nudge could recommended a course that is sequentially supposed to follow one more course.

As our nudge email is designed to look at the passed_timestamp and seemingly not much else, this overlooks important pieces like: has the course actually ended, has the assignment due date passed, have certificates been made available to learners in the course yet, is the learner in the course on the verified enrollment track in the course they just completed or are they an instructor or something else, etc.

@irfanuddinahmad irfanuddinahmad force-pushed the iahmad/ENT-6471 branch 3 times, most recently from ed3b21d to 61d19de Compare January 3, 2025 11:41
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.

1 participant