-
Notifications
You must be signed in to change notification settings - Fork 68
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
xAPI -Report on lines individually. #4
Comments
Do note that each line may have multiple blanks as well, and it might not be possible to understand each line if it stands on its own. We're actually considering to remove the multiple lines feature. They were there because we initially didn't use a wysiwyg text editor in this content type, but now there is really not reason to have that feature anymore, and most ppl only uses one line and creates several lines with the wysiwyg editor anyway. |
There would need to be one statement per blank in order to fit with the interaction component data structure. |
I think I found that xAPI supports multiple blanks. Is our current implementation wrong then? I saw that it was troublesome but not impossible to implement support for multiple correct answers for each blank though. |
Sorry yes, you are quite right and so is your documentation. My error. I didn't get the interaction component properties when I tested though. Has that documentation been implemented? Here's the statement I got:
|
That was strange. If you try the example on H5P.org you get it in your console so it is implemented yes. https://h5p.org/fill-in-the-blanks Does it show up in console in your test server? |
Waiting for feedback here... |
Hi, sorry for the slow reply. Can you remind me how to turn on statements in the console? I know you mentioned it in another thread but I can't remember which repo it was on. |
Found it So confirmed, I'm not getting interaction components in console on my local wordpress either. |
Ok, that probably doesn't prove anything since there might be other event listeners that runs before your's messing up the statement. Have you got the latest version of the H5P library? Not the H5P plugin for Wordpress, but the content types https://h5p.org/update-all-content-types Are you looking at the "answered" statements? Could you provide a link to the page where this content is so that I may debug it? |
I aim to check those questions later today or tomorrow. Unfortunately I can't give you access as I'm running this on localhost. I don't have a live Wordpress I can test this on. |
Thanks. I was unable to reproduce the problem with your code when running it in WP. If you turn of any xAPI related plugins on your site, does this still happen in the console? |
Yes, I'm afraid I still get the same statement with only 1 plugin activated, H5P. |
Just to make sure we're talking about the same thing, can you post the statement you're seeing? |
Hm.. well I'm throughly puzzled then! |
Yeah, I would be able to figure this out with access to the server :) |
I close this issue, and you could consider open a new one if you still think the issue is with H5P and not your setup. |
OK. |
I actually found the problem now. When "Automatically check answers after input" is on it works, when it isn't on and you press the check button the properties are missing. This has been fixed on master and will be released soon. The problem seems to be caused by a merge that went wrong. |
This content type contains one or more liens of text with a blank in them. Currently the content type reports "answered" for all lines together. If the lines were seen as a sub content type, they could have individual "answered" statements with interaction component data. This would include detail such as the learner's response, possible responses and score and success for each line.
The text was updated successfully, but these errors were encountered: