Skip to content

Commit

Permalink
PM-43593 added comment about unimplemented lineitems service.
Browse files Browse the repository at this point in the history
  • Loading branch information
mk-kialo committed Oct 2, 2024
1 parent fb59f5e commit 6e8bcf2
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions classes/lti_flow.php
Original file line number Diff line number Diff line change
Expand Up @@ -427,8 +427,12 @@ public static function add_grading_service(MessagePayloadBuilder $payloadbuilder
// This is the endpoint used by Kialo to get the line item details and post student scores.
"lineitem" => (new moodle_url('/mod/kialo/lti_lineitem.php', $serviceparams))->out(false),

// This endpoint is currently unused by Kialo and therefore not implemented. But if we end up implementing it,
// this is what it will be called.
// The lineitems (plural) endpoint is used by Kialo to look up line items by resource link ID
// only if the line item URL is not included in the launch data.
// Since our plugin always includes the line item URL in the launch data (in this very claim),
// Kialo never needs to use this endpoint.
// So this endpoint is currently not necessary and therefore not implemented.
// But if we end up implementing it, this is what it will be called.
"lineitems" => (new moodle_url('/mod/kialo/lti_lineitems.php', $serviceparams))->out(false),
]);
}
Expand Down

0 comments on commit 6e8bcf2

Please sign in to comment.