Skip to content

Commit

Permalink
add session id field (#5167)
Browse files Browse the repository at this point in the history
The xAPI statement for evaluated part attempts is missing the datashop_session_id field, which makes DataShop calculations in the cloud a bit harder to do. This PR adds it in as an extension.
  • Loading branch information
darrensiegel authored Oct 4, 2024
1 parent 02b4ad0 commit c019f13
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,8 @@ defmodule Oli.Analytics.XAPI.Events.Attempt.PartAttemptEvaluated do
out_of: out_of,
feedback: feedback,
date_evaluated: timestamp,
part_id: part_id
part_id: part_id,
datashop_session_id: session_id
},
%{
attempt_guid: page_attempt_guid,
Expand Down Expand Up @@ -95,7 +96,8 @@ defmodule Oli.Analytics.XAPI.Events.Attempt.PartAttemptEvaluated do
"http://oli.cmu.edu/extensions/activity_id" => activity_revision.resource_id,
"http://oli.cmu.edu/extensions/activity_revision_id" => activity_revision.id,
"http://oli.cmu.edu/extensions/part_id" => part_id,
"http://oli.cmu.edu/extensions/attached_objectives" => attached_objectives
"http://oli.cmu.edu/extensions/attached_objectives" => attached_objectives,
"http://oli.cmu.edu/extensions/session_id" => session_id
}
},
"timestamp" => timestamp
Expand Down

0 comments on commit c019f13

Please sign in to comment.