-
Notifications
You must be signed in to change notification settings - Fork 19
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 context.registration to statements #4
Comments
Hm... Should the "registration session" be the page visit, or should it be the php login session? In general, I think out policy for this plugin is that we can add as much information as we want. If there is something that is "controversial", then we can add that also, but with an option to turn it on and off or change behavior under an "advanced settings" tab in the settings. In this case, are there clear reasons why it should be the page visit and not the php session you think? Or should there be a way to change it? Apart from that, sounds like a good plan! Go ahead and send a pull request if you want! |
Ping? :) I'm considering closing this issue, just because open issues are an eyesore to me. @garemoko have you given this any thought? If not, we can close it for now, and then we can always re-open it later. |
Hi. I'll try and take a look at this and respond tomorrow. Sorry for the delay! |
Cool! |
Hi, sorry again for the delay. The registration should be page visit because the H5P content resets every time you refresh the page. If progress was bookmarked, then PHP session would make more sense. |
Makes sense |
Hi @garemoko Open source hacking = good karma, you know... :) |
@limikael ha! I won't discount that I might do this at some point, but lots of other priorities at the moment I'm afraid (some of them open source!) |
:) Do you have a particular use case or user story or such where this would be useful? If so, then maybe it will motivate me to actually getting around to implementing it... |
It would allow you to report on all activity on a single visit to a page e.g. an attempt at a quiz made up of questions on a page. |
We should add a context.registration to statements that represents a particular refresh of/visit to the page the H5P sits on.
This could be achieved by generating the UUID here using code like this and appending it to statements here.
This would help to enable the LRS to distinguish between attempts at the content. See the discussion here.
The text was updated successfully, but these errors were encountered: