Samvera Tech Call 2021-11-17
Meeting Logistics:
Time: 9:00am PDT / Noon EDT
Zoom Meeting URL: https://princeton.zoom.us/j/7739591625
(link will launch Zoom client – if you do not have Zoom, expand the instructions below)
Agenda (meeting notes below)
Review status of Hyrax analytics PR - https://github.com/samvera/hyrax/pull/5089 - @Juliet Hardesty
<add your agenda item here (your name)>
Moderator: @Thomas Scherz
Notetaker: @Juliet Hardesty
Attendees:
Sara G (Notch8)
@Max Kadel , UNC Chapel Hill
@Lea Ann Bradford (Notch8)
you!
Meeting Process
Notes
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 with that code (nothing removed or modified) but question about whether or not to remove this code as part of this PR
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) later?
need to test with someone using current/old statistics code to make sure 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
November 24 Tech Call staying on calendar
Thomas moderating
Sara note taking
Can also use Dev Congress time the following week if anything further needed for analytics PR review