Skip to content

Commit

Permalink
fix: pipeline steps fix
Browse files Browse the repository at this point in the history
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
  • Loading branch information
dyudyunov authored Jul 9, 2024
1 parent cf7420a commit 1d3ce30
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
3 changes: 0 additions & 3 deletions tutorretirement/patches/openedx-lms-common-settings
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,6 @@ RETIREMENT_STATES = [
'LOCKING_ACCOUNT',
'LOCKING_COMPLETE',

'RETIRING_EMAIL_LISTS',
'EMAIL_LISTS_COMPLETE',

'RETIRING_ENROLLMENTS',
'ENROLLMENTS_COMPLETE',

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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' ]

0 comments on commit 1d3ce30

Please sign in to comment.