Right now we are saving the to_option, the course_id, and the user_id in the CourseEmail object.  We should not need to pass them in as well.  (I assume that we put them in the CourseEmail so that there is some record of these values if we want to check what's happened in the past.  If we don't want to use them from the CourseEmail, then they shouldn't also be stored there because there's no guarantee that they're correct.)