search

adlnet / xAPIWrapper

Wrapper to simplify communication to an LRS
https://adlnet.gov/projects/xapi/
Apache License 2.0
219 stars 114 forks source link

Added support for result object. #125

Closed bharatpareek closed 5 years ago

bharatpareek commented 5 years ago

Added support for result property from xAPI Spec. It's an optional property that represents a measured outcome related to the Statement in which it is included.

vbhayden commented 5 years ago

Assumed you were mistaken as the Result object is so common, but every implementation I can find tacks it on after the fact. Surprised this wasn't in already, thanks!

vbhayden commented 5 years ago

Uglify didn't know how to handle the default argument value syntax of result=null, so that was changed to just be the typical JS version (actor, verb, object, result) where arguments default to undefined by default.

Updated the minified build also.

bharatpareek commented 5 years ago

Thank you. That was fast. :dancer: