Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-39479 Add support for metadata (or LRS Statements) to events
  3. SAK-39559

If LrsStatement set on event just return it

    XMLWordPrintable

    Details

    • Type: Sub-task
    • Status: CLOSED
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 11.2, 12.0
    • Fix Version/s: 11.5 [Tentative], 12.0
    • Component/s: Kernel
    • Labels:
      None
    • 11 status:
      Resolved

      Description

      Missed a change on KNL-1451 to actually return the LRS_Statement when it was set on the event!

      I feel like it should just return the the "statement" is "complete" rather than trying to use that as a base and fill in the other parts like the actor/verb/context.

      This may want to be revisited.

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

              depends on

              Feature Request - Functionality that you would like to see in Sakai. SAK-39479 Add support for metadata (or LRS Statements) to events

              • Minor - Minor loss of function, problem where easy workaround is present, or functionality which would be nice to have.
              • CLOSED
              is depended on by

              Sub-task - The sub-task of the issue SAK-39560 Allow for null actor (and probably others) in LearningResourceStore xapi service constructor

              • Critical - Crashes, loss of data, severe memory leak, need functionality ASAP in production.
              • CLOSED

              Feature Request - Functionality that you would like to see in Sakai. SAK-34817 Add additional events to samigo xapi for scoring

              • Major - Major loss of function or need for functionality in production in the near future.
              • Verified

                Activity

                  People

                  Assignee:
                  jonespm Matthew Jones
                  Reporter:
                  jonespm Matthew Jones
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration