Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix: Rename send_activation_email task (step 2/3)

Browse Source 
The old name is
`student.send_activation_email`.

The new name is
`common.djangoapps.student.tasks.send_activation_email`.

We currently register both the old and the new task names,
such that Celery workers recognize the task by both names.
This commit switches us from the old name to the new name.
This commit is contained in:
Kyle McCormick
2021-02-02 10:29:03 -05:00
parent b626a43fa1
commit 21adbc4b87
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
common/djangoapps/student/tasks.py
View File

@@ -78,8 +78,8 @@ _NEW_TASK_NAME = 'common.djangoapps.student.tasks.send_activation_email'
# Register task under both its old and new names,
# but expose only the old-named task for invocation.
# -> Next step: Once we deploy and teach Celery workers the new name,
# set `send_activation_email` to the new-named task.
send_activation_email = shared_task(bind=True, name=_OLD_TASK_NAME)(_send_activation_email)
shared_task(bind=True, name=_NEW_TASK_NAME)(_send_activation_email)
# but expose only the new-named task for invocation.
# -> Next step: Once we deploy and stop using the old task name,
# stop registering the task under the old name.
shared_task(bind=True, name=_OLD_TASK_NAME)(_send_activation_email)
send_activation_email = shared_task(bind=True, name=_NEW_TASK_NAME)(_send_activation_email)