Hello, I would like to report an issue regarding the Announcements Add-on logic. The Problem: When sending an announcement for a specific course, the email is sent to all users associated with the course, including students who have already completed the course (status finished). In most use cases, announcements are intended for active students only. Students who have finished the course should not receive these updates as it is perceived as spam. Technical Analysis: I have traced the issue to the following files: In class-lp-email-announcements.php (in announcement 4.1.2 plugin): The handle function calls $lp_course_db->get_user_ids_enrolled( $courseModel->get_id() );. In class-lp-course-db.php (in learnpress 4.3.1 plugin file) (The Root Cause): The function get_user_ids_enrolled executes a SQL query that explicitly includes users with the status finished. Code snippet from class-lp-course-db.php: AND (status = %s OR status = %s ) ", $course_id, LP_COURSE_CPT, 'enrolled', 'finished' // <--- This causes the issue ); My Request: Could you please update this logic in a future release? Ideally, we would like one of the following solutions: Better Default Logic: Only fetch users with status enrolled for announcements. A Filter Hook: Add a filter inside class-lp-email-announcements.php allowing developers to modify the $user_ids array before the emails are sent. This would allow us to filter out finished users ourselves without modifying core files. Current workaround is editing the core file class-lp-course-db.php, but this is not sustainable for updates. Looking forward to your reply. Kind regards, Elisa
