Skip to content

Commit

Permalink
fix: Reorder pipeline steps
Browse files Browse the repository at this point in the history
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

Co-authored-by: Florian Haas <florian@cleura.com>
  • Loading branch information
dyudyunov and fghaas committed Jul 31, 2024
1 parent be18b07 commit 8b6bcdc
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
* [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.
* [Bug fix] Run the retirement pipeline for course enrollments prior to that for forum posts.

## Version 3.3.1 (2024-07-03)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +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_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 8b6bcdc

Please sign in to comment.