yetanalytics / lrs

Protocols, specifications, and logic for building an xAPI Learning Record Store (LRS) in Clojure(Script).
https://www.yetanalytics.com/lrs
Apache License 2.0
4 stars 1 forks source link

Set `related` to `true` when filtering #44

Closed kelvinqian00 closed 3 years ago

kelvinqian00 commented 3 years ago

Before, when filtering on non-Actor agents or non-Object activities, the query would fail to retrieve statements correctly because related_agents and related_activities, respectively, were not set to true.

In this mini-PR, we set those two params to true automatically, even for Actor agents and Object activities. Partly because I'm not sure how to predicate on the agent/activity usage, but also because there may be cases where, for instance, an Object activity is also used as a Context activity in another statement. Thus, automatically setting related to true allows for maximum flexibility.