-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PADV-806: feat: add new content authoring event signals #108
Conversation
271568e
to
32aa46a
Compare
32aa46a
to
e498e5f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JuanDavidBuitrago Do we really need to back port all content authoring event signals?.
For me it is clear the need to do it for COURSE_CREATED, but I don't know about the others.
openedx/core/djangoapps/content_libraries/tests/test_content_libraries.py
Outdated
Show resolved
Hide resolved
@Jacatove I added the complete content authoring event signals because I had to update the openedx-events version, so I consider it is better to implement all of them to avoid problem using that version. |
I will let it as a draft to future work if it's useful. Here is the PR with COURSE_CREATED signal only: #109 |
Description
Use the following
OpenEdxPublicSignal
events triggered by content creation and editing.Testing instruction
vue/PADV-806
to use the necessary changesmake studio-logs | grep test_event:
Create a course
Check the log for
COURSE_CREATED
and the XBLOCK_UPDATED event dataShould see something like:
Jira issue
Other information
Reference PR edx-platform: openedx#32599
Reference PR openedx/events: openedx/openedx-events#244