Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

  • 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