xAPI-vle / moodle-logstore_xapi

A Moodle plugin to send xAPI statements to an LRS using events in the Moodle logstore.
GNU General Public License v3.0
74 stars 85 forks source link

Pseudonimisation and role assigned #836

Closed AD1529 closed 1 year ago

AD1529 commented 1 year ago

Hi,

I updated some files.

davidpesce commented 1 year ago

Thanks for contributing this; it seems like quite a bit of work!

It's missing the unit tests for each of the new events being monitored. Can you add those? If you are looking for an example, check out the tests folder in this repository.

In an ideal world, each new feature would be it's own PR. Is it possible to send them as separate PRs?

AD1529 commented 1 year ago

I'll open a new one