search

opencast-ilias / OpenCast

Opencast plugin for ILIAS LMS. The plugin shows a Opencast series as an ILIAS object. This version of the ILIAS plugin for Opencast is operated and developed collaboratively by a community. The University of Bern acts as coordinative maintainer.
GNU General Public License v3.0
13 stars 14 forks source link

[Release 8] Upgrade paella-user-tracking to v1.42.5 #336

Closed ferishili closed 3 months ago

ferishili commented 3 months ago

This PR fixes #323

It only contains the build changes after upgrading paella-user-tracking plugin to v1.42.5

ferishili commented 3 months ago

@snoesberger, please give it a try and write your feedback here.

Another question: is this change only for release_8?

snoesberger commented 3 months ago

I have successfully tested this PR.

Unfortunately the Opencast event ID is not available within the Paella player at the moment, only the video title (metadata.title) can be referenced in the paella player config. @ferishili as discussed in the Matrix chat, it would be really nice if the Opencast event ID could be made available.

ferishili commented 3 months ago

videoid as well as seriesid is now available in metadata:

il.Opencast.Paella.player.data.metadata.videoid il.Opencast.Paella.player.data.metadata.seriesid

snoesberger commented 3 months ago

Many thanks for the quick implementation! We were now able to successfully configure and test the paella player to sent video and series ID to Matomo. This is part of the config we used for our tests:

...
"es.upv.paella.matomo.userTrackingDataPlugin": {
            "enabled": true,
            "context": [
                "userTracking"
            ],
            "server": "https://matomo-test.org/",
            "siteId": "1",
            "trackerUrl": {
                "php": "matomo.php",
                "js": "matomo.js"
            },
            "matomoGlobalLoaded": false,
            "cookieType": "tracking",
            "mediaAnalyticsTitle": "${metadata.videoid}",
            "logUserId": true,
            "events": {
                "category": "ILIAS PaellaPlayer",
                "action": "${event}",
                "name": "${eventData}"
            },
            "customDimensions": {
                "1": "${metadata.videoid}",
                "2": "${metadata.seriesid}",
                "3": "${metadata.title}"
            }
        },
...

The statistic data is successfully sent to Matomo both for live streams and VoD.

dagraf commented 3 months ago

Since tests were successfull and @ferishili: Can you please prepare this PR for merging and then hand it over to @chfsx for a review and the actual merge? Thx!

ferishili commented 3 months ago

@chfsx the PR is ready to merge, no rebase is needed atm!

chfsx commented 3 months ago

Thanks a lot, merged to v8.1.4!