From 1d3ce3005e83bf543c4a6983f768d0dd3f3702dd Mon Sep 17 00:00:00 2001 From: Eugene Dyudyunov Date: Tue, 9 Jul 2024 14:22:24 +0300 Subject: [PATCH] fix: pipeline steps fix Retire mailings step uses LMS API that was removed starting from Ironwood release so it was removed. Retirement pipiline steps order was different comparing to retirement state indexes so the order was changed - enrollments now placed before forum. YT: - https://youtrack.raccoongang.com/issue/PhU-399 --- tutorretirement/patches/openedx-lms-common-settings | 3 --- .../retirement/build/retirement/pipeline_config/config.yml | 3 +-- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/tutorretirement/patches/openedx-lms-common-settings b/tutorretirement/patches/openedx-lms-common-settings index 3ad8504..0393685 100644 --- 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 index da9f7da..f256d37 100644 --- a/tutorretirement/templates/retirement/build/retirement/pipeline_config/config.yml +++ b/tutorretirement/templates/retirement/build/retirement/pipeline_config/config.yml @@ -6,9 +6,8 @@ base_urls: {% if NOTES_HOST is defined %}notes: {{ "https" if ENABLE_HTTPS else "http" }}://{{ NOTES_HOST }}{% endif %} retirement_pipeline: + - [ 'RETIRING_ENROLLMENTS', 'ENROLLMENTS_COMPLETE', 'LMS', 'retirement_unenroll' ] {% 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' ]