-
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
Add tracking logs #2
Add tracking logs #2
Conversation
Hi @andrey-canon ! I've (finally) started looking at how best to get your changes into the completion aggregator so that Aspects users can use these events. I'm sorry it's taken me so long! Do you have time to look at this PR in the next week or so? If not, that's totally fine -- I can resubmit it against master and get it reviewed by OpenCraft. cf openedx/openedx-aspects#222 for the plan so far, comments are very welcome. |
and adds CHANGELOG entry.
57b10f0
to
bde2bdf
Compare
Closed in favour of open-craft#206 |
Description:
This PR updates open-craft#173 to merge latest
master
and get tests passing again.open-craft#173 is the first step for openedx/openedx-aspects#222
JIRA:
Private-ref: FAL-3766
Merge deadline: ASAP -- there's follow-up work to come soon.
Installation instructions:
[completion.enable_completion_tracking](http://local.edly.io:8000/admin/waffle/switch/1/change/)
Testing instructions:
tail -f $(tutor config printroot)/data/lms/logs/tracking.log | grep compl
You should see
openedx.completion_aggregator.progress.*
events coming through to mark the progress towards completion of the units (vertical), subsections (sequential), sections (chapter), and course.When you've completed the children of these blocks, you should see corresponding
openedx.completion_aggregator.completion.*
events.Reviewers:
Merge checklist:
Post merge:
finished.