Comment 8 for bug 1517298

Revision history for this message
Jane Sandberg (sandbergja) wrote :

Thanks, Jeff, Dan, and Rogan. Rogan, I tried your branch, and it definitely worked. I was able to track a bunch of movements using a free trial Matomo dashboard -- very cool.

Before signing off, there are a few changes I'd want to see:
* Requiring 3 different permissions to get this thing rolling seems excessive to me. Could we please just use a single permission?
* On a similar note, I'd prefer to just have two org unit settings: opac.analytics.matomo_id and opac.analytics.matomo_url. Rather than checking for a third org unit setting, the tt2 could just check for the presence of these two org unit settings. This would reduce a step to get this feature running, and save an entry in an already huge list of org settings. Plus, it doesn't make much sense to enable this feature without a site id or a url. I could be convinced, though.
* Some light rebasing needed in the seed data, as always. :-)
* Can I propose some new description text for opac.analytics.matomo_id and opac.analytics.matomo_url?

  -> opac.analytics.matomo_id: The Site ID for your Evergreen catalog. You can find the Site ID in the tracking code you got from Matomo.
  -> opac.analytics.matomo_url: The URL for your the Matomo software. Be sure to include the trailing slash, e.g. https://my-evergreen.matomo.cloud/

I'd be happy to take these on, if there is agreement about them.