From eb1797d2102c04ddb34f3486af5dfed8da8ffe6c Mon Sep 17 00:00:00 2001 From: oliviaruizknott Date: Tue, 23 Nov 2021 10:59:46 -0500 Subject: [PATCH] feat: allow bulk email templates to be deleted MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit There are some old bulk email templates in the database that we don’t need anymore. We need to make this change in order to delete them. After removing the templates we want to remove, we’ll leave in the ability to delete. [MICROBA-1573] --- lms/djangoapps/bulk_email/admin.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/lms/djangoapps/bulk_email/admin.py b/lms/djangoapps/bulk_email/admin.py index 2599e5f334..478f44af25 100644 --- a/lms/djangoapps/bulk_email/admin.py +++ b/lms/djangoapps/bulk_email/admin.py @@ -57,19 +57,11 @@ unsupported tags will cause email sending to fail. ''' }), ) - # Turn off the action bar (we have no bulk actions) - actions = None def has_add_permission(self, request): """Enable the ability to add new templates, as we want to be able to define multiple templates.""" return True - def has_delete_permission(self, request, obj=None): - """ - Disables the ability to remove existing templates, as we'd like to make sure we don't have dangling references. - """ - return False - class CourseAuthorizationAdmin(admin.ModelAdmin): """Admin for enabling email on a course-by-course basis."""