Jlc/add common plugin tests #42
Conversation
ff081b7 to
df9cf0b
Compare
There was a problem hiding this comment.
Could you split this in two different PRs, celery task and command and could you do a rebase over the branch https://github.com/eduNEXT/eox-nelp/tree/and/add_notifications_models to start with the implementation?
eox_nelp/settings/common.py
Outdated
| settings.INSTALLED_APPS.append(COURSE_CREATOR_APP) | ||
|
|
There was a problem hiding this comment.
probably this will be no necessary
There was a problem hiding this comment.
you are meaning the white line?
There was a problem hiding this comment.
it fails me if I do not register the task(in the worker). At the moment the notifications task file is not called or imported in a principal file. So celery couldn't start a not registered task.
There was a problem hiding this comment.
That's why I said that probably this is not necessary, for reference check this PR I didn't need this
d95402c to
539c8a9
Compare
539c8a9 to
ba3dd23
Compare
Description
This PR has two principal functions.
Useful information to include:
Checklist for Merge