...
Sara G (Notch8)
Max Kadel , UNC Chapel Hill
Lea Ann Bradford (Notch8)
you!
...
Meeting Process
Expand | ||
---|---|---|
| ||
|
...
Review status of Hyrax analytics PR - https://github.com/samvera/hyrax/pull/5089 - Juliet Hardesty
Sara pushed new code to address comments, mostly documentation related
no other updates so far
comments regarding existing code pointing to “old way” (statistics way) of analytics - nothing changed there but question about whether or not to remove
User profile areas might show things differently but unclear if this means this code should be removed?
maybe a separate PR to remove old code (statistics)?
need to test someone using current/old statistics code to see if adding analytics PR as it is doesn’t break anything with statistics
Thomas Scherz will test this since they have statistics working
then later PR can remove statistics code as a way to encourage upgrading to use new analytics