fix: Drop reference to mailings retirement

Retire mailings step uses LMS API that was removed starting from Ironwood release so it was removed. Co-authored-by: Florian Haas <[email protected]>
hastexo · Jul 31, 2024 · be18b07 · be18b07
1 parent 2d3d233
commit be18b07
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,7 @@
 ## Unreleased
 
 * [Bug fix] Complete the removal of (not-ever-working) support for the Ecommerce service, by also removing references to that service from the `openedx-lms-common-settings` patch.
+* [Bug fix] Remove references to the mailing API (which in turn was removed from the LMS in Ironwood).
 * [Bug fix] Remove references to the (not-ever-working) Discovery service.
 
 ## Version 3.3.1 (2024-07-03)

diff --git a/tutorretirement/patches/openedx-lms-common-settings b/tutorretirement/patches/openedx-lms-common-settings
@@ -6,9 +6,6 @@ RETIREMENT_STATES = [
     'LOCKING_ACCOUNT',
     'LOCKING_COMPLETE',
 
-    'RETIRING_EMAIL_LISTS',
-    'EMAIL_LISTS_COMPLETE',
-
     'RETIRING_ENROLLMENTS',
     'ENROLLMENTS_COMPLETE',
 

diff --git a/tutorretirement/templates/retirement/build/retirement/pipeline_config/config.yml b/tutorretirement/templates/retirement/build/retirement/pipeline_config/config.yml
@@ -8,7 +8,6 @@ base_urls:
 retirement_pipeline:
     {% if FORUM_VERSION is defined %}- [ 'RETIRING_FORUMS', 'FORUMS_COMPLETE', 'LMS', 'retirement_retire_forum' ]{% endif %}
     {% if NOTES_HOST is defined %}- [ 'RETIRING_NOTES', 'NOTES_COMPLETE', 'LMS', 'retirement_retire_notes' ]{% endif %}
-    - [ 'RETIRING_EMAIL_LISTS', 'EMAIL_LISTS_COMPLETE', 'LMS', 'retirement_retire_mailings' ]
     - [ 'RETIRING_ENROLLMENTS', 'ENROLLMENTS_COMPLETE', 'LMS', 'retirement_unenroll' ]
     - [ 'RETIRING_LMS_MISC', 'LMS_MISC_COMPLETE', 'LMS', 'retirement_lms_retire_misc' ]
     - [ 'RETIRING_LMS', 'LMS_COMPLETE', 'LMS', 'retirement_lms_retire' ]