search

xAPI-vle / moodle-logstore_xapi

A Moodle plugin to send xAPI statements to an LRS using events in the Moodle logstore.
GNU General Public License v3.0
76 stars 84 forks source link

Trim attempt_reviewed statement to only include necessary parts #160

Closed davidpesce closed 6 years ago

davidpesce commented 6 years ago

attempt_reviewed statements are a bit verbose for what they are. Should be a simple statement of:

User reviewed attempt.

Also, remove the QuestionSubmitted recipe so that answers aren't submitted every time.
https://github.com/xAPI-vle/moodle-logstore_xapi/blob/516bf881c09ff2ddee43694edfb84cde42ebe4e5/lib/translator/src/Controller.php#L52

ryasmi commented 6 years ago

In the new architecture branch I've started stripping a lot of stuff from statements, mostly extensions and descriptions.

On Thu, 22 Feb 2018, 19:32 David Pesce, notifications@github.com wrote:

attempt_reviewed statements are a bit verbose for what they are. Should be a simple statement of:

User reviewed attempt.

Also, remove the QuestionSubmitted recipe so that answers aren't submitted every time.

https://github.com/xAPI-vle/moodle-logstore_xapi/blob/516bf881c09ff2ddee43694edfb84cde42ebe4e5/lib/translator/src/Controller.php#L52

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/xAPI-vle/moodle-logstore_xapi/issues/160, or mute the thread https://github.com/notifications/unsubscribe-auth/AC54iRWxEUVDX2aGTVS-yLuw-7vjJQRXks5tXcDIgaJpZM4SP28a .

binote00 commented 6 years ago

We want to use logstore_xapi plugin in our Moodle projects and possibly modify the plugin. Your architecture branch looks very interesting especially when you said "stripping a lot of stuff from statements, mostly extensions and descriptions" What's your plans about this branch ? Any ETA ?

ryasmi commented 6 years ago

https://github.com/xAPI-vle/moodle-logstore_xapi/releases/tag/v3.0.0